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

Add run.sh script to project #2517

Merged
merged 1 commit into from
Apr 11, 2024
Merged

Add run.sh script to project #2517

merged 1 commit into from
Apr 11, 2024

Conversation

jonaharagon
Copy link
Member

Changes proposed in this PR:

  • Adds a run.sh script for running the website locally. The switch to environment variables in Remove per-language config files #2513 probably made it harder to run translated copies locally, now you can: ./run.sh --lang=fr
  • Switches from git committers plugin to git authors plugin, because the git committers plugin wasn't caching things right and I wasn't really a fan of the GitHub-specific nature of it anyways (this is the plugin that adds authors to each page's footer FYI)
  • Adds .mailmap to merge multiple git identities from authors in git log / git blame - I added the most obvious suspects I could find at a glance, but we should strive to add every person who commits with multiple Git identities over time to this file.
  • I have disclosed any relevant conflicts of interest in my post.
  • I agree to grant Privacy Guides a perpetual, worldwide, non-exclusive, transferable, royalty-free, irrevocable license with the right to sublicense such rights through multiple tiers of sublicensees, to reproduce, modify, display, perform, relicense, and distribute my contribution as part of this project.
  • I am the sole author of this work.
  • I agree to the Community Code of Conduct.

@jonaharagon jonaharagon requested a review from dngray as a code owner April 11, 2024 04:45
Copy link

github-actions bot commented Apr 11, 2024

Your preview is ready!

Name Link
🔨 Latest commit 2e787fd
😎 Preview https://2517--glowing-salamander-8d7127.netlify.app/

@jonaharagon
Copy link
Member Author

fyi mailmap is also needed for https://timvink.github.io/mkdocs-git-authors-plugin/mailmap.html it isn't just for git log

@jonaharagon jonaharagon marked this pull request as ready for review April 11, 2024 05:55
@jonaharagon jonaharagon added the ci:build i18n Enable translated builds on a PR label Apr 11, 2024
jonaharagon added a commit that referenced this pull request Apr 11, 2024
jonaharagon added a commit that referenced this pull request Apr 11, 2024
@dngray dngray merged commit c4c68f7 into main Apr 11, 2024
13 checks passed
@dngray dngray deleted the run-script branch April 11, 2024 06:42
@privacyguides-bot
Copy link
Collaborator

This pull request has been mentioned on Privacy Guides. There might be relevant details there:

https://discuss.privacyguides.net/t/2024-04-11/17822/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:build i18n Enable translated builds on a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants