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

IDEX-3576: Add git remote and branch to projects attributes #602

Merged
merged 1 commit into from
Jan 12, 2016

Conversation

vinokurig
Copy link
Contributor

@vparfonov
Copy link
Contributor

ok

@@ -22,11 +22,17 @@
public class GitProjectType extends TransientMixin {

public static final String VCS_PROVIDER_NAME = "vcs.provider.name";
public static final String CURRENT_BRANCH_NAME = "current.branch.name";
public static final String REPOSITORY_REMOTES = "repository.remotes";
Copy link
Contributor

Choose a reason for hiding this comment

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

Formatter

Copy link
Contributor

Choose a reason for hiding this comment

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

Actually this variant is also possible, but agree alligment is preferable.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@skabashnyuk
Copy link
Contributor

ok


@Inject
public GitProjectType(GitValueProviderFactory gitRepositoryValueProviderFactory) {
super("git", "git");
addVariableDefinition(VCS_PROVIDER_NAME, "Is this git repo or not?", false,
gitRepositoryValueProviderFactory);
addVariableDefinition(CURRENT_BRANCH_NAME, "Name of current brunch", false,
Copy link
Contributor

Choose a reason for hiding this comment

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

Fix "brunch"

@sleshchenko
Copy link
Contributor

other OK

vinokurig pushed a commit that referenced this pull request Jan 12, 2016
IDEX-3576: Add git remote and branch to projects attributes
@vinokurig vinokurig merged commit 01fe237 into master Jan 12, 2016
@vinokurig vinokurig deleted the IDEX-3576 branch January 12, 2016 08:34
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.

6 participants