Skip to content

feat: allow anonymous github emails#4

Open
shpingalet007 wants to merge 6 commits intoeErbLiddle:mainfrom
shpingalet007:feat/allow-anon-flag
Open

feat: allow anonymous github emails#4
shpingalet007 wants to merge 6 commits intoeErbLiddle:mainfrom
shpingalet007:feat/allow-anon-flag

Conversation

@shpingalet007
Copy link

@shpingalet007 shpingalet007 commented May 28, 2023

Introduction

Thank you, @evvanErb for your repository. I work with CI/CD in our team project, and your action does a good job.

A few months ago, I switched to an anonymous GitHub email address. However, it was after that I tried your repository. And when I went back a few days ago, I found that it wasn't working for me. After some investigation, I saw that you limited the action to not include anonymous GitHub email addresses. I assume the module was created for other use cases, but I would like to promote this PR.

I need my anonymous email address to be able to push automatically a commit that is authored by me. So it doesn't matter to me if it's a real address or not.

Changes

  • Added the option to allow anonymous GitHub email addresses to be captured.
  • Introduce a new flag allow-anonymous, set to false by default

Other changes

  • Introduce package scripts: build:clean and build
  • Install development dependency @vercel/ncc for build script
  • The build script produces a minified result. It reduced the size of the bundle by 30%.
  • Describe the input and output as tables in the README. My opinion is that this is better.
  • Removed redundant dist/xhr-sync-worker.js

@shpingalet007 shpingalet007 force-pushed the feat/allow-anon-flag branch from b0662ae to 2521bb9 Compare May 28, 2023 16:28
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.

1 participant