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 "save" command #127

Merged
merged 3 commits into from Nov 15, 2018
Merged

Add "save" command #127

merged 3 commits into from Nov 15, 2018

Conversation

tatyanaboloban
Copy link

Copy link
Contributor

@extsoft extsoft left a comment

Choose a reason for hiding this comment

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

@tatyanaboloban please take a look for the following:

  1. Squash "Commands list update" and "Add "save" command" commits
  2. Don't duplicate commit's subject and body in "Add test for 'git elegant save' command"
  3. "Update README.md" is a bad commit subject. Try to highlight what task it solves.
  4. The last line in the commit's body has to be a ticket number. See 4613c66 like an example.

docs/README.md Outdated
@@ -52,6 +52,9 @@ Shows trailing whitespaces of uncommitted changes.
### commands
Displays all available commands.

### save
Saves the changes to the current repository as a commit.
Copy link
Contributor

Choose a reason for hiding this comment

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

Use branch instead of repository.


default(){
boxtee git add --interactive
boxtee git elegant check --staged
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't use boxtee for git elegant commands.

Suggested change
boxtee git elegant check --staged
git elegant check --staged

Tatyana Boloban added 3 commits November 14, 2018 15:01
Add a description of the "save" command to the README.md

 bees-hive#113
Add "save" command which saves the changes to the current
branch as a commit
Add new command "save" to git-elegant-commands

 bees-hive#113
@extsoft
Copy link
Contributor

extsoft commented Nov 15, 2018

@rultor please merge.

@rultor
Copy link

rultor commented Nov 15, 2018

@rultor please merge.

@extsoft OK, I'll try to merge now. You can check the progress of the merge here

@rultor
Copy link

rultor commented Nov 15, 2018

@rultor please merge.

@extsoft Done! FYI, the full log is here (took me 2min)

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.

None yet

3 participants