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

Conda env attempt #33

Merged
merged 17 commits into from
Nov 5, 2024
Merged

Conda env attempt #33

merged 17 commits into from
Nov 5, 2024

Conversation

ilaflott
Copy link
Member

@ilaflott ilaflott commented Oct 4, 2024

this isn't supposed to be an all-encompassing conda env for all future fre-workflows functioning. it's more about having a functional-enough pipeline for SOME testing and linting feedback across-the-board.

something is always better than nothing.

ilaflott and others added 17 commits October 4, 2024 11:41
…flow things. another workflow yaml gutted from fre-cli for pipeline that lints cylc files and runs data_lineage pytest to start.
…nto workflow file and pytest call for make-timeseries
…s for most. expecting to likely not have all the req deps
…t_conda_env.yml edit). ignore files ending in *tags created by papiex_tooler tests. adjust to some more pythonic conventions inother test scripts
@ilaflott ilaflott linked an issue Oct 7, 2024 that may be closed by this pull request
@ilaflott ilaflott marked this pull request as ready for review October 11, 2024 00:27
@ilaflott
Copy link
Member Author

currently the workflows are all allowed to succeed on error. i thought it kind for now to give the tests a chance to catch up, but still get the "data" of testing logs going for reference, easy/mindless-feedback, and issue minting.

also i think merging wouldn't be allowed with a big red X, so...

@ilaflott ilaflott linked an issue Oct 29, 2024 that may be closed by this pull request
Copy link
Contributor

@ceblanton ceblanton left a comment

Choose a reason for hiding this comment

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

Thanks @ilaflott ! This is clearly in the right direction, and helpful to @cwhitlock-NOAA whichever way she goes

@ceblanton ceblanton merged commit 0a9837a into main Nov 5, 2024
1 check passed
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.

Add unit tests for the Jinja2 filters initial testing workflow efforts
2 participants