Skip to content

Tweak dependency formatting#1939

Merged
speth merged 1 commit intoCantera:mainfrom
ischoegl:tweak-dependencies
Aug 12, 2025
Merged

Tweak dependency formatting#1939
speth merged 1 commit intoCantera:mainfrom
ischoegl:tweak-dependencies

Conversation

@ischoegl
Copy link
Copy Markdown
Member

@ischoegl ischoegl commented Aug 11, 2025

Changes proposed in this pull request

Ensure consistent formatting and fix some typos

Checklist

  • The pull request includes a clear description of this code change
  • Commit messages have short titles and reference relevant issues
  • Build passes (scons build & scons test) and unit tests address code coverage
  • Style & formatting of contributed code follows contributing guidelines
  • The pull request is ready for review

@ischoegl ischoegl requested a review from a team August 12, 2025 00:41
Copy link
Copy Markdown
Member

@speth speth left a comment

Choose a reason for hiding this comment

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

Thanks, @ischoegl. Just a couple of minor suggestions.

@ischoegl ischoegl force-pushed the tweak-dependencies branch from 8ba3578 to 2e95987 Compare August 12, 2025 12:28
@ischoegl
Copy link
Copy Markdown
Member Author

Thanks, @speth - I had checked that the links are active, but didn't realize that there were now some redirects. Thanks for catching that!

@ischoegl ischoegl requested a review from speth August 12, 2025 14:00
@speth speth merged commit af8298c into Cantera:main Aug 12, 2025
49 of 50 checks passed
@ischoegl ischoegl deleted the tweak-dependencies branch August 12, 2025 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants