Skip to content
This repository was archived by the owner on Aug 11, 2025. It is now read-only.

Update README.md#305

Merged
dougiesquire merged 1 commit intomainfrom
cbull/readmeupdate
Aug 11, 2025
Merged

Update README.md#305
dougiesquire merged 1 commit intomainfrom
cbull/readmeupdate

Conversation

@chrisb13
Copy link
Contributor

@chrisb13 chrisb13 commented Aug 8, 2025

Tiny tweaks prompted by this PR (ACCESS-NRI/ACCESS-OM3#136) from @anton-seaice.

@anton-seaice
Copy link
Contributor

I would prefer if this repository was archived - see ACCESS-NRI/access-om3-configs#563

@chrisb13
Copy link
Contributor Author

chrisb13 commented Aug 8, 2025

Yeah ok, I'll follow up in that other thread but if we do archive, the README will still be visible so as such I think this PR would be best merged before then?

@dougiesquire
Copy link
Collaborator

Do we want to add this before I archive or not? I'm not sure that adding links (which may move/break) is a good idea. It might be better to remove links?

@chrisb13
Copy link
Contributor Author

chrisb13 commented Aug 8, 2025

I think we might as well. The old version has links too it's just that they're now a little out of date.

On the other hand, if there's a strong view of no links, then they should be removed.

@dougiesquire
Copy link
Collaborator

That's kinda my point. The old links weren't added that long ago and one already doesn't work. Once we archive, we won't be able to modify (without unarchiving) so adding more links that might break is maybe not a good idea.

@chrisb13
Copy link
Contributor Author

chrisb13 commented Aug 8, 2025

Well, I don't mind removing. I think the earlier link in that file: https://github.com/ACCESS-NRI/access-om3-configs is unlikely to go anywhere.

Having said that, the one you allude to was a more detailed link and the new ones are higher up. I think the Hive Docs folks have been talking about a re-factor so I'm not sure how long those links will last.

@anton-seaice
Copy link
Contributor

I think deleting all the content except maybe the logos, and link to https://github.com/ACCESS-NRI/access-om3-configs only?

Alternatively, When its archived, can we write a message in the header? e.g. just say see https://github.com/ACCESS-NRI/access-om3-configs for latest information and leave the Readme as is

@dougiesquire
Copy link
Collaborator

I think just remove the "Documentation (including building and running the model)" section. The other section already refers to https://github.com/ACCESS-NRI/access-om3-configs

@chrisb13
Copy link
Contributor Author

chrisb13 commented Aug 8, 2025

I think just remove the "Documentation (including building and running the model)" section. The other section already refers to https://github.com/ACCESS-NRI/access-om3-configs

Sounds good to me.

@dougiesquire dougiesquire force-pushed the cbull/readmeupdate branch 2 times, most recently from eaf6f80 to f25ce0b Compare August 11, 2025 22:40
@dougiesquire
Copy link
Collaborator

I've made the change. @chrisb13 / @anton-seaice could you please review

Co-authored-by: Christopher Bull <5499680+chrisb13@users.noreply.github.com>
Copy link
Contributor

@anton-seaice anton-seaice left a comment

Choose a reason for hiding this comment

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

Thanks @dougiesquire

@dougiesquire dougiesquire merged commit 5b1c9a7 into main Aug 11, 2025
2 checks passed
@dougiesquire dougiesquire deleted the cbull/readmeupdate branch August 11, 2025 23:02
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.

3 participants