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

Update supplementary github files to improve repo organization and maintenance #2463

Merged
merged 2 commits into from
Jul 18, 2018

Conversation

dlstadther
Copy link
Collaborator

Description

  • Adds .github/ directory for organization
  • Adds configuration for Stale Probot

Motivation and Context

The luigi repo has been getting cluttered overtime from abandoned feature requests, bug reports, help requested, and code submissions. The goal of adding Stale is not to keep ignoring contributors' interactions, but rather to keep those of recent relevance and active addition in focus.

There are two sides to make this an effective solution to help improve the quality (both actual and perceived) of spotify/luigi.

  1. contributors need to actively follow their PRs/Issues from submission to merge/close (myself included)
    • it's ok if you don't have time to continue working on them - it's fine to close and return to them at a later date
    • perhaps we need to create a label for those items which will be completed, just at a later date
  2. maintainers need to provide feedback at quicker intervals to allow for less stale PRs/Issues (again, myself included)
    • perhaps we add a .github/CODEOWNERS to auto tag certain maintainers or other frequently active contributors to aid in the code review speed up process
    • if others are in favor, i'd be happy to add this file with primary maintainers to start

Have you tested this? If so, how?

No testing needed - supplementary github files

@dlstadther dlstadther changed the title Feature/stale bot Update supplementary github files to improve repo organization and maintenance Jul 17, 2018
Copy link
Contributor

@Tarrasch Tarrasch left a comment

Choose a reason for hiding this comment

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

Awesome!

@Tarrasch Tarrasch merged commit 20d7e3f into spotify:master Jul 18, 2018
@dlstadther dlstadther deleted the feature/stale-bot branch July 18, 2018 20:35
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.

2 participants