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

♻️ Refactor Actions to accept email for GitLab #46

Merged

Conversation

amadeu01
Copy link
Member

@amadeu01 amadeu01 commented Sep 23, 2020

Closes: #27

Note:

  • we do not handle the case where the user attempts to add its username/email multiple times;
  • we are not validating email formats.

@amadeu01 amadeu01 added the enhancement New feature or request label Sep 23, 2020
@amadeu01 amadeu01 self-assigned this Sep 23, 2020
@@ -0,0 +1,20 @@
defmodule GitPair.Storage do

Choose a reason for hiding this comment

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

Modules should have a @moduledoc tag.

Copy link
Member Author

Choose a reason for hiding this comment

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

We will do it later.

@amadeu01 amadeu01 added this to the v1.0.0 milestone Sep 23, 2020
@amadeu01 amadeu01 force-pushed the ac-wt/accept-user-email-in-add-command branch from bc80fce to f84a7dc Compare September 23, 2020 01:36
@wevtimoteo wevtimoteo modified the milestones: v1.0.0, v1.1.0 Sep 29, 2020
Amadeu 🦄 Lindo 🌈 added 2 commits October 6, 2020 21:13
This module will be responsible to coordinate all interactions with `git config` command.
@amadeu01 amadeu01 force-pushed the ac-wt/accept-user-email-in-add-command branch from f84a7dc to 3675765 Compare October 7, 2020 00:18
Amadeu 🦄 Lindo 🌈 and others added 5 commits October 6, 2020 22:07
Co-authored-by: wevtimoteo <[email protected]>
For now, we are only using the `Storage` module for adding users. However, we are planning to refactor `Actions` to remove all `System` calls.

Co-authored-by: wevtimoteo <[email protected]>
Amadeu 🦄 Lindo 🌈 and others added 2 commits October 6, 2020 22:18
Co-authored-by: wevtimoteo <[email protected]>
Co-authored-by: wevtimoteo <[email protected]>
@sourcelevel-bot
Copy link

SourceLevel has finished reviewing this Pull Request and has found:

  • 1 possible new issue (including those that may have been commented here).

See more details about this review.

@amadeu01 amadeu01 marked this pull request as ready for review October 7, 2020 01:26
@wevtimoteo wevtimoteo changed the base branch from main to add-support-email-from-pairs October 29, 2020 23:48
@amadeu01 amadeu01 merged commit 2c2f609 into add-support-email-from-pairs Oct 30, 2020
@amadeu01 amadeu01 deleted the ac-wt/accept-user-email-in-add-command branch October 30, 2020 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Determine co-author anonymous email by remote origin address
2 participants