-
Notifications
You must be signed in to change notification settings - Fork 8
GEM-21 update/remove jiscSD references #852
base: main
Are you sure you want to change the base?
Conversation
| @@ -1,2 +1,2 @@ | |||
| # The codeowners for the Octopus repository are | |||
| * @JiscSD/Octopus-Dev | |||
| * @Jisc/Octopus-Dev | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this might be a breaker..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or we just ignore it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this will never make any sense really, we either remove it or use @Jisc (which wont work in jisc-SDG), the side effect is PRs wont automatically get assigned to the team, or we update it once merged and remove it from Jisc/octopus (that will give a conflict though and the repos wont be exact copies)
| <img src="https://www.jisc.ac.uk/sites/all/themes/jisc_clean/img/jisc-logo.svg" align="right" width=50 height=50/><h1 align="left">Octopus</h1> | ||
|
|
||
|  | ||
|  |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
broken on branch but I think will work when merged to main (though for Jisc/octopus it will still be broken or will say 'skipped')
| Some code has been removed that is no longer needed for the ongoing operation of octopus, but is logged here for future reference: | ||
|
|
||
| - Seed data scripts that were used to process the initial data used to populate the database. They were last included in [this commit](https://github.com/JiscSD/octopus/tree/fdd0fc5e0f673a1fe42c4cec0b74fe126be2225b/seed-data-scripts). | ||
| - Seed data scripts that were used to process the initial data used to populate the database. They were last included in [this commit](/tree/fdd0fc5e0f673a1fe42c4cec0b74fe126be2225b/seed-data-scripts). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
again, link will work when merged to main
karmseveeratjisc
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wont merge until migrated anyway
295416a to
d3b220c
Compare
The purpose of this PR was...
DONT MERGE until migrated to github enterprise