Skip to content

ci: update config to reflect core repo#988

Merged
sdepold merged 3 commits into
mainfrom
update-ci
Jan 14, 2022
Merged

ci: update config to reflect core repo#988
sdepold merged 3 commits into
mainfrom
update-ci

Conversation

@WikiRik
Copy link
Copy Markdown
Member

@WikiRik WikiRik commented Jan 11, 2022

Pull Request check-list

Please make sure to review and check all of these items:

  • Does npm run test pass with this change (including linting)?
  • Does the description below contain a link to an existing issue (Closes #[issue]) or a description of the issue you are solving?
  • Have you added new tests to prevent regressions?
  • Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?

Description of change

Updated the CI config so it is more aligned with the core repo. Do note that v6 and v7 branches do not exist yet, I will make a v6 branch after #987 and this PR have been merged. v7 branch will follow at a later stage.

@WikiRik WikiRik requested review from ephys and sdepold January 11, 2022 14:39
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml
@sdepold
Copy link
Copy Markdown
Member

sdepold commented Jan 13, 2022

I just added the OpenCollective and Twitter Secrets

@WikiRik WikiRik requested a review from sdepold January 14, 2022 12:31
@WikiRik
Copy link
Copy Markdown
Member Author

WikiRik commented Jan 14, 2022

@sdepold I think the comments are resolved, can you approve?

@sdepold sdepold merged commit f04c1a5 into main Jan 14, 2022
@sdepold sdepold deleted the update-ci branch January 14, 2022 13:10
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 6.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants