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

Improvement: add gitee and jsdelivr resource url #371

Merged
merged 3 commits into from
Sep 5, 2020

Conversation

nooop3
Copy link
Contributor

@nooop3 nooop3 commented Jun 26, 2020

To resolve the GitHub access issue in some regions, I added a new environment variable HARDCODE_FIXER_SOURCE, which can set to gitee and jsdelivr and default is github. Can temporarily address #369 until the fallback is added.

@Foggalong Foggalong linked an issue Jun 27, 2020 that may be closed by this pull request
@Foggalong
Copy link
Owner

@nooop3 I like this as a temporary solution, but it doesn't close #369 since I've made that about creating an offline fallback option so that it covers wider issues of network access. It does need some more work though so that it better integrates into the script; will do a more thorough review this weekend.

@nooop3
Copy link
Contributor Author

nooop3 commented Jun 27, 2020

@Foggalong Yes, you are right. This is a temporary solution. And I only added the new_date check to avoid the script error, maybe we could add another offline flag, so we can skip the upgrade check in the offline situation.

@Foggalong
Copy link
Owner

@nooop3 For sure, you did the right thing bumping the date! And yeah as discovered over there the plan is to add that kind of fallback :)

Original change by @nooop3 had it assigned as an environment variable, 
but probably more useful to have them directly checked as fallbacks. 
This also sets up the infrastructure to have the local version as the 
ultimate fallback.
@Foggalong
Copy link
Owner

Updated your code to make those fallbacks available directly through the script rather than as env variables. Thanks so much for getting the ball rolling on this 😄

@Foggalong Foggalong merged commit 0b84acf into Foggalong:master Sep 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Offline CSV Fallback Option
2 participants