Skip to content

Conversation

@third774
Copy link
Member

I've got a Bluesky account set up for us, and this PR should provide the validation needed for them to set the handle to austinjavascript.com

This moves us onto dart sass which can run on apple silicon chips
@third774
Copy link
Member Author

Had to move us from node sass to dart sass (so it runs on apple silicon), hope that's okay!

Also, LMK if I should back out the changes to the _cache folder. Pretty sure these updated when I ran the build locally. Should the folder be removed and added to .gitignore?

Copy link
Member

@joahg joahg left a comment

Choose a reason for hiding this comment

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

yeah let's .gitignore the _cache files please :) other than that, lgtm!

  • add .gitignore for _cache

@joahg joahg merged commit e28ad1b into austinjavascript:master Nov 15, 2024
@stedman
Copy link
Member

stedman commented Dec 3, 2024

Also, LMK if I should back out the changes to the _cache folder. Pretty sure these updated when I ran the build locally. Should the folder be removed and added to .gitignore?

Hey Kevin, I apologize for the _cache directory — it was a source of pain back when i set this up. There were a lot of legacy Twitter images that I manually cobbled together in that directory and then put under source control to prevent Github Pages from overwriting. I was hoping to get back to it with a better solution but Covid ended the meetups and suddenly the site didn't seem like highest priority anymore.

You'll find the smoking gun here: https://github.com/austinjavascript/austinjavascript.com/blob/master/_includes/shortcodes/avatar.js#L15-L17

Happy to help with any questions y'all might have.

@third774
Copy link
Member Author

third774 commented Dec 3, 2024

Ahhhhhh, thanks for the context! It does look like a bunch of profile pics are broken! I'll open a PR to revert it. PR #181
CleanShot 2024-12-03 at 00 46 47@2x

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.

3 participants