You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The reason will be displayed to describe this comment to others. Learn more.
@samsrabin it looks like there is an error in this line now when running GitHub Actions. See error below from one of my commits (on my personal branch).
[Invalid workflow file: .github/workflows/assign-to-project.yml#L16](https://github.com/TeaganKing/CTSM/actions/runs/7804055717/workflow)
You have an error in your yaml syntax on line 16
It also looks like I can't submit a PR with workflow changes, so I thought I'd just tag you. I'm not sure if it's the dash that's an issue now? Maybe putting this in quotes would help?
The reason will be displayed to describe this comment to others. Learn more.
@TeaganKing we brought an update on master (but not in any tags) that is working for me. I'm having to update my branches
to escomp/master and then it seems to fix the problem. Have you tried that? The one thing I find is that I have to do this with
any branch I work with, so the more branches I touch the more I have to update.
829da76
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@samsrabin it looks like there is an error in this line now when running GitHub Actions. See error below from one of my commits (on my personal branch).
It also looks like I can't submit a PR with workflow changes, so I thought I'd just tag you. I'm not sure if it's the dash that's an issue now? Maybe putting this in quotes would help?
829da76
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ugh! Thanks for the heads up.
829da76
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TeaganKing we brought an update on master (but not in any tags) that is working for me. I'm having to update my branches
to escomp/master and then it seems to fix the problem. Have you tried that? The one thing I find is that I have to do this with
any branch I work with, so the more branches I touch the more I have to update.
829da76
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When I put the corrected code on master in a YAML syntax checker it passes with the dash above.
One example is:
https://yamlchecker.com/
829da76
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I see! Sorry for the false alarm @samsrabin , and thanks for letting me know about that update @ekluzek !
829da76
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No problem. I'd rather have a culture where everyone feels free to point out issues, than one where everyone is too afraid to do that!