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

Conversation

@StevenClontz
Copy link
Collaborator

@StevenClontz StevenClontz commented Jan 18, 2024

Addresses #138

@siwelwerd siwelwerd merged commit a4d8400 into main Jan 18, 2024
@AbbyANoble
Copy link
Contributor

AbbyANoble commented Jan 24, 2024

Question -- does merged and committed mean it's fixed on the live version? Because I'm not seeing it if so.

@siwelwerd
Copy link
Collaborator

It means the source code has been fixed. There is a separate "deploy" step that builds and publishes the HTML on the website. Looking at the front matter, it was deployed in December, before this fix, so that's why you are not seeing it.

Generally we avoid any deploys mid-semester because we don't want to change numbering or something on someone mid-semester while they are teaching with it.

@AbbyANoble
Copy link
Contributor

Gotcha. So many words that mean "fix"! haha.

@StevenClontz
Copy link
Collaborator Author

I need to fix this repo so we can deploy a "2024 Edition PREVIEW". Will do this now...

@StevenClontz StevenClontz mentioned this pull request Jan 24, 2024
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.

4 participants