search

archspec / archspec-json

Other
20 stars 32 forks source link

updated additions for nvhpc compilers #66

Closed cparrott73 closed 1 year ago

cparrott73 commented 1 year ago

I have reworked these based on feedback from the archspec maintainers.

cparrott73 commented 1 year ago

Hold on, I'm aware of the JSON failures, working on a fix.

alalazo commented 1 year ago

@cparrott73 Can you please rebase? @glennpj Would you have some time to test drive this one again?

alalazo commented 1 year ago

Uh, what happened here?

cparrott73 commented 1 year ago

Sorry - I must have botched this somehow. I am not very experienced with git and was trying to figure out how to rebase and merge my changes. I will respin this.

alalazo commented 1 year ago

@cparrott73 You can push to the same branch on your repo, and use that instead of master to submit the PR. Also, any change you do to your repo can be force pushed in case of errors - so no need to reopen PRs in general. Let me know if you need further help :slightly_smiling_face: