-
Notifications
You must be signed in to change notification settings - Fork 7
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
Config repos unsearchable / broken wiki links #75
Comments
Hmm, that's quite annoying. Personally I wouldn't change the main branch. I think the reasons for leaving it empty are good ones. Also, it might be misleading to have a search that only works on one configuration, ignoring all the others. I see two other options:
I think I favor the first option, but all have pros and cons... |
I think it would work quite well to set Re. your first dot point, did you have something in mind as a way to search/document non-default branches in the configs? It can be done offline with re. 2nd dot point, we wanted to avoid splitting into multiple repos to make it easier to maintain multiple configs. I think this consideration more than compensates for the annoying Github limitations on searching other branches. |
What about adding some config files in the main branch to fix the wiki documentation, but no |
It looks like you can search branches in github.dev
(ref https://stackoverflow.com/questions/31891733/searching-code-in-a-specific-github-branch) |
Wow, that's cool! But is it possible to see the results of a search via a url like this? I'd need that for links. |
Oh i see the problem now ... not possible in anyway I can see |
Unfortunately I don't think that would work. The wiki currently searches in these files (which includes config.yaml), and we'd probably want to search more as the documentation takes shape.
I think having so many files (including config.yaml) in 'main' would be more confusing than just using an actual config, and more of a maintenance burden to keep it synced with the working config branch it's based on. |
@aekiss I think this could still work if all the files, including the config.yaml file, are put into a subdirectory (e.g. Edit: this would still require manual update of the files. On the other hand, it would allow to list more options than those that are used in the |
Ah, good idea, let's do that and see how we go. |
See if this PR fits the bill ACCESS-NRI/access-om3-configs#36 Still need to update links in wiki. |
Closing - I've added the necessary files to We should run At some point we may want to switch to basing the wiki documentation on MOM6-CICE-WW3, so this PR will facilitate that ACCESS-NRI/access-om3-wav-configs#16 |
GitHub only searches the default branch, so shifting the config content out of the main branch has made the configs unsearchable.
This has also broken a whole lot of links in the wiki (e.g. https://github.com/COSIMA/access-om3/wiki/Configurations), which were mostly GitHub search queries.
I'm not sure what to do about this. Being unable to search the branches via the UI is a real hindrance to understanding and documenting the configs.
Maybe the default branch should be something useable, e.g.
1deg_jra55do_ryf
rather than just a README?The text was updated successfully, but these errors were encountered: