Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test with new julia LTS 1.10 #53

Closed
wants to merge 2 commits into from
Closed

Conversation

maxstb
Copy link
Contributor

@maxstb maxstb commented Jan 8, 2025

Github actions are running with the new API (#46) and new secrets.
For the secrets however it's important to store it in the following way:
CDSAPI_URL = "url: https://cds.climate.copernicus.eu/api"
CDSAPI_KEY = "key: XXXXXXXXXXXXXXXXXXXXX"

I just considered that we should update the tests to the new LTS version of julia

Fix #46

@maxstb
Copy link
Contributor Author

maxstb commented Jan 8, 2025

Just changed the secret usage to only store the values in github instead of the key-value-pair

@maxstb
Copy link
Contributor Author

maxstb commented Jan 8, 2025

@juliohm can you check the secrets again?
CDSAPI_URL = "https://cds.climate.copernicus.eu/api"
CDSAPI_KEY = "XXXXXXXXXXXXXXXXXXXXX"
They should look like this. The CI in my fork runs fine with this setup

@@ -17,7 +17,7 @@ jobs:
fail-fast: false
matrix:
version:
- '1.6'
- '1.10'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- '1.10'
- 'lts'

Copy link
Member

@juliohm juliohm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The keys are in the expected format, without the prefix. I've made the LTS support explicit in the CI.yml.

@juliohm
Copy link
Member

juliohm commented Jan 8, 2025

I can't propose changes in this branch of your fork, but I am assuming that CI will succeed after you accept the suggestion.

@juliohm
Copy link
Member

juliohm commented Jan 8, 2025

Tried these changes on the master branch. Let's see if they solve the issue.

@juliohm
Copy link
Member

juliohm commented Jan 8, 2025

Issue fixed on master ❤️ Thank you @maxstb

@juliohm juliohm closed this Jan 8, 2025
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.

Fix build on GitHub Actions
2 participants