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.

Sign up for GitHub

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

Jump to bottom

follow up of deleting realizations, #1488

Merged
merged 1 commit into from
Nov 30, 2023
Merged

follow up of deleting realizations, #1488

merged 1 commit into from
Nov 30, 2023

Conversation

LaviniaBaumstark
Copy link
Member

Purpose of this PR

rename cm_peakBudgYr -> cm_peakBu…dgYr and cm_taxCO2inc_after_peakBudgYr -> c_taxCO2inc_after_peakBudgYr as thei are no interfaces any more

Type of change

  • Bug fix

Checklist:

  • My code follows the coding etiquette
  • I performed a self-review of my own code
  • All automated model tests pass (FAIL 0 in the output of make test)

Further information (optional):

  • Test runs are here:
  • Comparison of results (what changes by this PR?):

@LaviniaBaumstark
follow up of deleting realizations, rename cm_peakBudgYr -> cm_peakBu…
f15c003 
…dgYr and cm_taxCO2inc_after_peakBudgYr -> c_taxCO2inc_after_peakBudgYr as thei are no interfaces any more
@LaviniaBaumstark LaviniaBaumstark merged commit 4aa6793 into remindmodel:develop Nov 30, 2023
2 checks passed
@orichters orichters mentioned this pull request Dec 1, 2023
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q 0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q 0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LaviniaBaumstark @0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q