Skip to content
This repository was archived by the owner on Nov 12, 2024. It is now read-only.

Apply scm policy #558

Merged
merged 1 commit into from
Sep 9, 2024
Merged

Apply scm policy #558

merged 1 commit into from
Sep 9, 2024

Conversation

yngve793
Copy link
Contributor

@yngve793 yngve793 commented Jan 3, 2024

Update in accordance with Equinor SCM policy.

Prepare README.md for project rename.

Copy link
Contributor Author

@yngve793 yngve793 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was suggested to add the standard SCM policy files before archiving the project. I am aware that several of these changes do not make sense at this point in time, but here we are. It is unclear to me how much work we should put into the project before archiving it.

I suggest to push your "[Prototype] Version tests 2.0" and then the basic SCM before archive.

@achaikou
Copy link
Contributor

Prototype PR should be closed I think. I hardly remember what it was about.

There is no such thing as standard SCM policy. Policy code that we submit should be fixed, at least factually incorrect things.
If we are archiving the project it doesn't mean that we can allow ourselves to push there anything incorrect.

Though we won't bother with fixing CI as this would actually require putting work into project which we don't need.

Copy link
Contributor Author

@yngve793 yngve793 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed the raised issues.
I am not sure if we want to merge non working code.

Copy link
Contributor

@achaikou achaikou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now we'll just have not to forget to fix here all the links to oneseismic before renaming!

@yngve793 yngve793 merged commit ca9cc8b into equinor:master Sep 9, 2024
0 of 8 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants