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

Allow to group commits by scope #48

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

danielgomezrico
Copy link

By I deleted the original repo error I closed this PR because :

Allow to group commits by scope

Fix:

Done:

  • Use Case: when all commits have scopes
  • Use Case: some commits don't have scopes
  • Fix Other work is indentated inside Other work
  • Use Case: Breaking Change
  • Fix rubocop offenses

Current

# Current
- **User Login:** add User Income Form Components 
- **Map:** search without country/state filters to open map borders to other cities/countries
- **Map:** revert -> search without country/state filters to open map borders to other cities/countries

The preview:

  • User Login: add User Income Form Components
  • Map: search without country/state filters to open map borders to other cities/countries
  • Map: revert -> search without country/state filters to open map borders to other cities/countries

Grouping

# Expected
- **User Login:** add User Income Form Components 
- **Map:**
   - search without country/state filters to open map borders to other cities/countries
   - revert -> search without country/state filters to open map borders to other cities/countries

The preview:

  • User Login: add User Income Form Components
  • Map:
    • search without country/state filters to open map borders to other cities/countries
    • revert -> search without country/state filters to open map borders to other cities/countries

@danielgomezrico danielgomezrico changed the title Feature/add group by scope Allow to group commits by scope Aug 27, 2024
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