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

bugfix git commit #258

Closed
wants to merge 529 commits into from
Closed

bugfix git commit #258

wants to merge 529 commits into from

Conversation

abhimishr
Copy link
Member

Purpose of this PR

There are instances of "missing" gms package on windows machines. Changes from this PR will make sure that gms is installed if it is not found during commit. Might need at-least one instance of running git commit with elevated rights on windows machines.

Performance loss/gain from current default behavior

NA

Type of change

  • Bug fix (Change which fixes an issue).

How Has This Been Tested?

NA

Additions or Changes to default configuration (default.cfg):

NA

Checklist:

NA

Special comments/warnings

Commits directly via IDEs like atom probably would take much longer as git commit now always(?) calls a codeCheck (the background process is not visible in these IDEs).

bodirsky and others added 30 commits January 20, 2021 09:15
changelog
renamed and shortened setnames
added some comments
removed typos
# Conflicts:
#	modules/15_food/anthropometrics_jan18/input.gms
#	modules/15_food/anthropometrics_jan18/presolve.gms
flohump and others added 27 commits February 27, 2021 09:45
… ovoid overspecialization of plantation establishment towards highly productive cells.
…nd s35_hvarea = 2., otherwise infeasible (Upper bound in trade module).
Reshuffling of forestry, natveg and timber module code : Better routine for timber production inclusing performance updates
Automatic sets in 56_ghg_policy and 60_bioenergy and update of GHG emission policies
updated pre-commit hook to check all git managed files
added file size check to github actions
tweaked pre-commit hook and github actions workflow
@abhimishr
Copy link
Member Author

Wrong branch! sorry

@abhimishr abhimishr closed this Mar 9, 2021
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.

None yet

5 participants