Skip to content

Conversation

@webwarrior-ws
Copy link
Contributor

@webwarrior-ws webwarrior-ws commented Nov 6, 2025

To the repository root so it can be referenced and used in Github workflows. The action is originally from [1] and is
a workaround to problem outlined in [2].

Use "dynamic uses" action in workflow to specify version of actions/checkout only once and avoid duplication.

[1] https://github.com/jenseng/dynamic-uses
[2] https://github.com/orgs/community/discussions/25246

@knocte
Copy link
Member

knocte commented Nov 6, 2025

The action is originally from [1].

Please also reference the github issue where we saw his comment.

To the repository root so it can be referenced and used in
Github workflows. The action is originally from [1] and is
a workaround to problem outlined in [2].

Contains failing CI.

[1] https://github.com/jenseng/dynamic-uses
[2] https://github.com/orgs/community/discussions/25246
Otherwise nonVerboseFlagsInGitHubCIAndScripts.fsx will
complain.
To specify version of actions/checkout only once and avoid
duplication.
Action with nblockchain/conventions@main. This commit
contains failing CI, but it should work once PR is merged.
@knocte
Copy link
Member

knocte commented Nov 6, 2025

Can you rename action.yml to dynamc-uses-action.yml and place it inside .github dir? Would that work?

@webwarrior-ws
Copy link
Contributor Author

Can you rename action.yml to dynamc-uses-action.yml and place it inside .github dir? Would that work?

That would work for this repo but I don't think such action can be referenced from other repository.

@knocte
Copy link
Member

knocte commented Nov 6, 2025

That would work for this repo but I don't think such action can be referenced from other repository.

Because of the rename or because of placing it in .github dir?

@webwarrior-ws
Copy link
Contributor Author

That would work for this repo but I don't think such action can be referenced from other repository.

Because of the rename or because of placing it in .github dir?

Because it is no longer action.yaml file in the repository root.

@knocte
Copy link
Member

knocte commented Nov 6, 2025

Ok let's see, merging.

@knocte knocte merged commit 639d425 into nblockchain:master Nov 6, 2025
3 of 5 checks passed
webwarrior-ws added a commit to webwarrior-ws/conventions that referenced this pull request Nov 6, 2025
To include the correct branch name (master).

This should have been fixed in [1].

[1] nblockchain#207
knocte pushed a commit that referenced this pull request Nov 6, 2025
To include the correct branch name (master).

This should have been fixed in [1].

[1] #207
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.

2 participants