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

Transferring upload permission to "sat_blazemeter" #590

Merged
merged 1 commit into from Feb 16, 2018
Merged

Transferring upload permission to "sat_blazemeter" #590

merged 1 commit into from Feb 16, 2018

Conversation

dzmitrykashlach
Copy link
Contributor

@dzmitrykashlach dzmitrykashlach commented Feb 13, 2018

Transferring upload permissions from zmicerkashlach to sat_blazemeter for
BlazeMeterJenkins plugin

@oleg-nenashev
Copy link
Contributor

@dzmitrykashlach have you logged in to Artifactory from that account as suggested in the checklist?

@daniel-beck
Copy link
Contributor

They have not.

The checklist for PRs to this repo exists for a reason. Please review it, and complete all items that apply.

@daniel-beck daniel-beck added needs fix needs initial login The users need to log in to Jira and/or Artifactory labels Feb 13, 2018
@daniel-beck
Copy link
Contributor

@dzmitrykashlach
Copy link
Contributor Author

@daniel-beck logged in to Artifactory with sat_blazemeter user. Please, check.

@daniel-beck daniel-beck removed the needs initial login The users need to log in to Jira and/or Artifactory label Feb 13, 2018
@artem-fedorov
Copy link

@oleg-nenashev @daniel-beck
Could you please describe which steps we skipped?

@daniel-beck
Copy link
Contributor

Could you please describe which steps we skipped?

https://github.com/jenkins-infra/repository-permissions-updater/blob/master/.github/PULL_REQUEST_TEMPLATE.md exists for a reason; it makes the job of the not-even-a-handful people who process these requests much easier. So I've decided to no longer process requests that don't even bother to do everything described there, even if it's as simple as…

Add link to plugin/component Git repository in description above

@dzmitrykashlach
Copy link
Contributor Author

dzmitrykashlach commented Feb 14, 2018

@daniel-beck @oleg-nenashev
Sorry for misunderstanding.
I've updated initial comment to PR with link to
BlazeMeterJenkinsPlugin github repository
Thank you for assistance.

@daniel-beck daniel-beck merged commit 6b7575b into jenkins-infra:master Feb 16, 2018
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

4 participants