Closed Bug 1520083 Opened 7 years ago Closed 7 years ago

Remove unused third party access from github org

Categories

(Taskcluster :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bstack, Assigned: bstack)

Details

In the next few days I want to go through and remove a lot of things from [0].

Can we form a list of the things we still use that we should keep?

From my end of the world it is probably:

  • heroku dashboard
  • netlify auth
  • travis

Anyone have any others?

[0] https://github.com/organizations/taskcluster/settings/oauth_application_policy

Current list of approved things:

AppVeyor
AppVeyor CI
Atom GitHub package
Codecov
Coveralls
Docker Hub Builder
drone.io
Greenkeeper
Heroku Dashboard
Netlify Auth
Rollbar
Sentry
Snyk
Travis CI for Open Source
Waffle

Assignee: nobody → bstack
Status: NEW → ASSIGNED
Flags: needinfo?(wcosta)
Flags: needinfo?(pmoore)
Flags: needinfo?(jhford)
Flags: needinfo?(helfi92)
Flags: needinfo?(dustin)
Flags: needinfo?(coop)
Flags: needinfo?(bugzeeeeee)

I might have been the one who approved Codecov - it can be removed then. Greenkeeper seems to duplicate Renovate's functionality. I don't recognize the rest of the apps

Flags: needinfo?(bugzeeeeee)
  1. Renovate

I like renovate more than greenkeeper because it's more flexible with scheduling and can even send PRs on a weekly basis to update yarn.lock which is pretty neat.

  1. Snyk
  2. Sentry
  3. Travis
  4. Heroku Dashboard
Flags: needinfo?(helfi92)

travis, heroku and sentry are things I think we should keep, but they already been requested.

Flags: needinfo?(jhford)

As mentioned in IRC, waffle can be disabled. No one is using it.

Flags: needinfo?(coop)

I think the sentry thing in there is actually for when we used hosted sentry and not for the one that mozilla runs.

appveyor we can ditch - in the end, it was only used by https://github.com/taskcluster/runlib but that repo got merged into generic-worker and was archived.

Flags: needinfo?(pmoore)

Generic-worker uses coveralls.

Otherwise, nothing here that hasn't been mentioned that I want to keep.

Flags: needinfo?(dustin)
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: needinfo?(wcosta)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.