-
Notifications
You must be signed in to change notification settings - Fork 71
chore: skip commit verification + bump @babel/* dependencies #642
Conversation
b10491f
to
428e28a
Compare
thanks! |
Pre-release failed... unsure what the problem is here https://github.com/marvinpinto/actions/actions/runs/5249691334 |
Investigating... |
I'm assuming a PAT token expired? GITHUB_SUPER_TOKEN? @marvinpinto otherwise there is no fix if we can't cross push. We need admin access to edit the child repos to allow this master repo to have actions native permissions instead of PAT. looks like a block without marvin |
@sbe-arg @marvinpinto yes, it's a blocker, I cannot help here |
emailed @marvinpinto with details if there is no input we can probably fork everything in a new org with a few maintainers |
@sbe-arg any update? |
Because the cross repository sharing permissions are broken and can only be fixed by @marvinpinto we are limited in how to implement the builds. I don't want to re engineer the flows much. I asked @Enase to disable linting wich is breaking the checks after each pr and each push to main |
@sbe-arg no reason to disable linters, just fix the issues and that's it. see the very last comment on PR |
Id prefer if we disable the linters for the commits altogether |
@sbe-arg If you disable the commit linter, it will prevent the |
@sbe-arg as a result we need to deal with changelog generation. release data wont |
Changes:
actions/checkout
git action updatedyarn-cache
action replaced withactions/setup-node@v3
github action (cache configured)Update Github Runner Node version and deprecate set-output
message added to ignore list forcommitlint
(produced by Update Github Runner Node version and deprecate set-output #546)@babel/*
dependencies updatedprettier
version updatedtypescript
version set in order to meet dependencies requirementsfixes: #546