Skip to content

221.Add force options for run-fremake tool #340

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
Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

221.Add force options for run-fremake tool #340

wants to merge 18 commits into from

Conversation

singhd789
Copy link
Collaborator

@singhd789 singhd789 commented Feb 3, 2025 •
edited
Loading

Describe your changes

Issue ticket number and link (if applicable)

Checklist before requesting a review

  • I ran my code
  • I tried to make my code readable
  • I tried to comment my code
  • I wrote a new test, if applicable
  • I wrote new instructions/documentation, if applicable
  • I ran pytest and inspected it's output
  • I ran pylint and attempted to implement some of it's feedback

Dana Singh and others added 17 commits February 3, 2025 14:25
#221 Add force options for run-fremake tool
c41e3ad
Merge branch 'main' of github.com:NOAA-GFDL/fre-cli into 221.run-for…
aca9bc9 
…ce-opts
#221 Fix run-fremake test and arguments
b65ea1b
#221 Update run_fremake_script
0bb3260
#221 Add no_format_transfer to fremake.py for run-fremake
f1accea
#221 Merge branch 'main' of github.com:NOAA-GFDL/fre-cli into 221.ru…
3582596 
…n-force-opts
#221 Fix run-fremake.py
3fa5fa8
#221 Update fremake_parallel to run
bb14afd
#221 Move execute bit up
ffd0a69
#221 Revert back to fremake_parallel
99e764a
#221 Fix spelling
0b534f2
#221 Attempt at fixing run-fremake with force options
a591dd0
#221 Fix default parallel option
c7e6636
#221 Fix where execute option is
412e671
#221 Fix default value (change from False to 1)
245fc17
#221 Remove comments
c0bbd95
#221 Create OUT folder for gitignore to catch fre/make/test out …
a478072 
…folders
@singhd789 singhd789 marked this pull request as ready for review April 21, 2025 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rem1776 rem1776 Awaiting requested review from rem1776

@uramirez8707 uramirez8707 Awaiting requested review from uramirez8707

@thomas-robinson thomas-robinson Awaiting requested review from thomas-robinson

At least 1 approving review is required to merge this pull request.

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

Successfully merging this pull request may close these issues.
1 participant
@singhd789