Skip to content

updated the pre-commit configuration#63

Merged
norlandrhagen merged 1 commit intoProjectPythia:mainfrom
Anu-Ra-g:precommit_update
Aug 29, 2024
Merged

updated the pre-commit configuration#63
norlandrhagen merged 1 commit intoProjectPythia:mainfrom
Anu-Ra-g:precommit_update

Conversation

@Anu-Ra-g
Copy link
Copy Markdown
Contributor

@Anu-Ra-g Anu-Ra-g commented Aug 26, 2024

This PR contains new pre-commit configuration.

@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 38bdc55
✅ Deployment Preview URL: In Progress

@martindurant
Copy link
Copy Markdown
Contributor

PR #64 is blocked until this is merged, so would appreciate someone having a look here.

@norlandrhagen norlandrhagen merged commit 292956a into ProjectPythia:main Aug 29, 2024
@martindurant
Copy link
Copy Markdown
Contributor

Thanks @norlandrhagen .

@martindurant
Copy link
Copy Markdown
Contributor

@Anu-Ra-g : the build following merging this PR failed, please check
https://github.com/ProjectPythia/kerchunk-cookbook/actions/runs/10617299100/job/29429761379

@martindurant
Copy link
Copy Markdown
Contributor

(I can access the file from local now, so it might have been an intermittent network effect)

@Anu-Ra-g
Copy link
Copy Markdown
Contributor Author

@martindurant The same issue has also been raised here in #61. I'm running the notebook associated with this error.

@martindurant
Copy link
Copy Markdown
Contributor

NoCredentials implies that it is trying to authenticate - but the data is public, so anon=True should be in the remote_options.

@Anu-Ra-g
Copy link
Copy Markdown
Contributor Author

Yeah, I noticed in the notebook, while opening the reference filesystem remote_options are not provided. I'll run it and make the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants