Skip to content

doc - document CI information#1975

Open
jeremylt wants to merge 1 commit into
mainfrom
jeremy/doc-ci
Open

doc - document CI information#1975
jeremylt wants to merge 1 commit into
mainfrom
jeremy/doc-ci

Conversation

@jeremylt
Copy link
Copy Markdown
Member

Purpose:

More CI documentation for @zatkins-dev. Congratulations on being good at stuff :)

Closes: #N/A

LLM/GenAI Disclosure:

None

By submitting this PR, the author certifies to its contents as described by the Developer's Certificate of Origin.
Please follow the Contributing Guidelines for all PRs.

@jeremylt
Copy link
Copy Markdown
Member Author

A no-code label might be handy for libCEED like we have with Ratel


When managing the installed dependencies, use the `phypid` account.

Note: PETSc commit `7ada2a5134eeb4113b6693316dd2660902f08a96` must be reverted for libCEED fluid dynamics examples to pass (see [issue 1686](https://github.com/CEED/libCEED/issues/1686)).
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Is there a git patch for this? If not, it should be easy to set up so that we aren't relying on the commit hash.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

we don't have a patch file, no. We can create one by running git revert 7ada2a5134eeb4113b6693316dd2660902f08a96?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is more something @jrwrigh knows the ins and outs on

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Huzza, I guessed it was poor memory management somewhere or another but hadn't had the time to find out where

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