mono
mono copied to clipboard
refactor: updated `actions/github-script`
Proposed changes
Resolves https://github.com/db-ui/mono/issues/2375
Types of changes
- [ ] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Refactoring (fix on existing components or architectural decisions)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
- [ ] Documentation Update (if none of the other choices apply)
Further comments
🔭🐙🐈 Test this branch here: https://db-ui.github.io/mono/review/2375-chore-updated-actionsgithub-script
@nmerget there are two new features based on the changelog – as you're much more aware of the pipeline implementation, is there something that we need to further change on this PR?
@nmerget there are two new features based on the changelog – as you're much more aware of the pipeline implementation, is there something that we need to further change on this PR?
It's fine, the changes won't affect us besides node20
@nmerget there are two new features based on the changelog – as you're much more aware of the pipeline implementation, is there something that we need to further change on this PR?
It's fine, the changes won't affect us besides node20
@nmerget okay, in other words, would we profit somehow by adapting our code to these new features, or wouldn't they be of any need for us?
@nmerget there are two new features based on the changelog – as you're much more aware of the pipeline implementation, is there something that we need to further change on this PR?
It's fine, the changes won't affect us besides node20
@nmerget okay, in other words, would we profit somehow by adapting our code to these new features, or wouldn't they be of any need for us?
We don't have the need, if the pipeline runs it's okay. We can check in the future if we can improve it somehow
@nmerget there are two new features based on the changelog – as you're much more aware of the pipeline implementation, is there something that we need to further change on this PR?
It's fine, the changes won't affect us besides node20
@nmerget okay, in other words, would we profit somehow by adapting our code to these new features, or wouldn't they be of any need for us?
We don't have the need, if the pipeline runs it's okay. We can check in the future if we can improve it somehow
why not today? what would be (another) trigger in the future ?
@nmerget there are two new features based on the changelog – as you're much more aware of the pipeline implementation, is there something that we need to further change on this PR?
It's fine, the changes won't affect us besides node20
@nmerget okay, in other words, would we profit somehow by adapting our code to these new features, or wouldn't they be of any need for us?
We don't have the need, if the pipeline runs it's okay. We can check in the future if we can improve it somehow
why not today? what would be (another) trigger in the future ?
Because we want to release a beta version? If you have time for it, do it ;)
@nmerget there are two new features based on the changelog – as you're much more aware of the pipeline implementation, is there something that we need to further change on this PR?
It's fine, the changes won't affect us besides node20
@nmerget okay, in other words, would we profit somehow by adapting our code to these new features, or wouldn't they be of any need for us?
We don't have the need, if the pipeline runs it's okay. We can check in the future if we can improve it somehow
why not today? what would be (another) trigger in the future ?
Because we want to release a beta version? If you have time for it, do it ;)
I've moved the target for this update to the next release after Beta.
fixed with https://github.com/db-ui/mono/pull/2837