Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update of NOR Manager SOP #2571

Closed
wants to merge 3 commits into from
Closed

Update of NOR Manager SOP #2571

wants to merge 3 commits into from

Conversation

pfabry
Copy link
Contributor

@pfabry pfabry commented Apr 9, 2024

No description provided.

@cmungall
Copy link
Contributor

can you normalize on newlines so it's easier to see changes?

@nlharris
Copy link
Contributor

can you normalize on newlines so it's easier to see changes?

Oh wow, yeah, what happened there that the entire document is shown as being changed? You think that's different flavors of newlines, Chris?

@nlharris
Copy link
Contributor

@pfabry since the way you did the edits make it look to GitHub like every single line has changed, can you (1) avoid doing editing that way in the future, and (2) explain what you actually did change so we can comment on that? Thanks!

matentzn added a commit that referenced this pull request Aug 10, 2024
@matentzn
Copy link
Contributor

I have recreated this PR here: https://github.com/OBOFoundry/OBOFoundry.github.io/pull/2631/files

@nlharris nlharris closed this Aug 11, 2024
nlharris added a commit that referenced this pull request Aug 20, 2024
* Update RoleNORManager.md

Replaces #2571

* Apply suggestions from code review

Co-authored-by: Nomi Harris <[email protected]>

* Update docs/roles/RoleNORManager.md

Co-authored-by: Nomi Harris <[email protected]>

* Update docs/roles/RoleNORManager.md

Co-authored-by: Nomi Harris <[email protected]>

---------

Co-authored-by: Nomi Harris <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants