Skip to content
This repository has been archived by the owner on Mar 1, 2023. It is now read-only.

Use Thermodynamics package #2209

Merged
merged 1 commit into from Jun 7, 2021
Merged

Use Thermodynamics package #2209

merged 1 commit into from Jun 7, 2021

Conversation

charleskawczynski
Copy link
Member

Description

This PR makes ClimateMachine.jl use the registered Thermodynamics.jl package.

  • Code follows the style guidelines OR N/A.
  • Unit tests are included OR N/A.
  • Code is exercised in an integration test OR N/A.
  • Documentation has been added/updated OR N/A.

@charleskawczynski charleskawczynski force-pushed the ck/use_thermo_pkg branch 3 times, most recently from d7fa4a6 to 4bb4324 Compare June 4, 2021 21:42
@charleskawczynski charleskawczynski mentioned this pull request Jun 4, 2021
Merged
@charleskawczynski charleskawczynski marked this pull request as ready for review June 7, 2021 18:53
@charleskawczynski
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jun 7, 2021

Build succeeded:

@bors bors bot merged commit 2def0d6 into master Jun 7, 2021
@bors bors bot deleted the ck/use_thermo_pkg branch June 7, 2021 20:41
@charleskawczynski charleskawczynski mentioned this pull request Jul 19, 2021
Merged
4 tasks
bors bot added a commit that referenced this pull request Jul 20, 2021
@bors @charleskawczynski
Merge #2219
74964c5 
2219: Remove Thermodynamics module r=charleskawczynski a=charleskawczynski

### Description

This was meant to be included in #2209.



Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kpamnany kpamnany kpamnany approved these changes

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@charleskawczynski @kpamnany