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

Pull in changes from 'upstream' #39

Open
wants to merge 90 commits into
base: main
Choose a base branch
from
Open

Conversation

CallumWalley
Copy link
Member

@CallumWalley CallumWalley commented Sep 27, 2024

Sorry, meant to do this days ago, got sidetracked.

Let me know if you have any questions, changes, etc.

Changes:

  • Moving from branch-based gh-pages to action-based gh-pages.
  • Switch from literate-nav to awesome-pages (this is the cause of most of the changes. Had to find-replace quite a lot). .pages.yml files now do the same function as summary.md, more details. I left most of the explicitly set names in place, but it's best to use the filename when possible.
  • Offsite links will be opened in new tab by default.
  • Adds 'comment on this page' button next to 'edit this page'.
  • Removed homepage from top navbar.
  • Redirect map file (currently empty, but may be needed).
  • 'Get support' button in nav, links to https://support.cloud.nesi.org.nz/.
  • Added NeSI favicon.
  • Renamed 'assets/css/' to assets/stylesheets (seems slightly more common and is what is used on other sites).
  • Added requirements.in for use with pip compile.
  • Change 'table of contents' to 'on this page'.
  • Hide TOC when less than 2 entries.
  • Linting checks.
  • Added .vscode workspace settings.
  • Auto merge automation. (anything with label auto_merge will be merged at midnight)
  • Deploy test site on PR request.
  • Google analytics. (although won't start recording till after merge).
  • Added a little bit of colour (feel free to change back).

@lbrick
Copy link
Collaborator

lbrick commented Sep 29, 2024

@lbrick
Copy link
Collaborator

lbrick commented Sep 30, 2024

Basic checks looks good to me

Copy link
Collaborator

@lbrick lbrick left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants