frequency-chain / frequency

Frequency: A Polkadot Parachain
https://www.frequency.xyz
Apache License 2.0
48 stars 18 forks source link

Update Repo #1961

Closed wilwade closed 2 months ago

wilwade commented 2 months ago

Goal

The goal of this PR is to update the GitHub org of the repository

Discussion

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

wilwade commented 2 months ago

Besides org name changes and linting, I see some removed logic related to the ci-base-image. Could you comment on the purpose of that in relation to the org move?

@JoeCap08055 It was noted in the PR where it went, but the reason why is:

  1. The container registry images didn't move over, so we had to publish the images before being able to run any PRs
  2. We had wanted to get out of the double PR requirement and disconnect the image publishing from the normal CI flow.