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

Dependabot Config File #2379

Merged
merged 2 commits into from
Jul 23, 2024
Merged

Conversation

engelhartrueben
Copy link
Collaborator

Fixes # (issue)

Dependabot had no means of configuration. Added dependabot.yaml to .github/. I am guessing after the switch from MoveOn to StateVoices this was lost.

Description

Option Config Context
Ecosystem npm Checks package.json and any lock files against the npm registry.
Target Branch stage-main-14.1 This sets up the target branch. Needs to be changed after every update
Interval weekly Self explanitory
Day Monday ^^
Time 12:00 UTC / 08:00 EST ^^
Rebase-strategy auto One of many strategies. auto checks for changes to target branch, and rebases as needed
PR Limit 10 Default is 5, didn't think 10 would be an issue
Labels A-dependecies Adds a label to the PR. May want to change this in the future after we go through the many labels

Please see dependabot config documentation.

Effectively, every Monday 8am EST time, we should see PR's open from dependabot.

Checklist:

  • I have manually tested my changes on desktop and mobile
  • The test suite passes locally with my changes
  • If my change is a UI change, I have attached a screenshot to the description section of this pull request
  • My change is 300 lines of code or less, or has a documented reason in the description why it’s longer
  • I have made any necessary changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • My PR is labeled [WIP] if it is in progress

@engelhartrueben engelhartrueben requested a review from mau11 July 17, 2024 15:11
@engelhartrueben
Copy link
Collaborator Author

Actually, this may want to go to main so the target-branch configuration works

@mau11
Copy link
Member

mau11 commented Jul 17, 2024

@engelhartrueben I agree since we want the bot to continuously run, not just for 14.1

we can change the target branch to main and merge this to main

@engelhartrueben
Copy link
Collaborator Author

engelhartrueben commented Jul 17, 2024

@mau11 target-branch is the "Branch to create pull requests against". So all PR's from dependabot would pull against stage-main-14.1 with this current configuration.

I was more getting at this PR should be merged into main so dependabot uses this .yaml file and sets the target-branch to stage-main-14.1

@engelhartrueben engelhartrueben changed the base branch from stage-main-14.1 to main July 19, 2024 13:09
@engelhartrueben
Copy link
Collaborator Author

rabased for main

@engelhartrueben engelhartrueben changed the base branch from main to stage-patch-main-14.0.1 July 23, 2024 21:39
@engelhartrueben engelhartrueben merged commit 813c1a6 into stage-patch-main-14.0.1 Jul 23, 2024
17 checks passed
@engelhartrueben engelhartrueben deleted the re/dependabot-config branch July 23, 2024 21:43
@engelhartrueben engelhartrueben mentioned this pull request Jul 24, 2024
7 tasks
engelhartrueben added a commit that referenced this pull request Jul 24, 2024
* bug/2382: currentInteractionStep variable needs to continue w/in block instead of new const

* Dependabot Config File (#2379)

* init dependabot.yml

* typo

* fix for campaign sort bug (#2384)

* fix bug where the CampaignTable would white screen of death after trying to sort

* set initial sort to ID_DESC

* remove sorting on archive column

* call this.state.campaigns directly wherever possible. Object destructured pageInfo immidiately

* add 14.0.1 release notes

* 14.0 >> 14.0.1

* link pr's

* fix typo

---------

Co-authored-by: Schuyler Duveen <sduveen@pivotbio.com>
Co-authored-by: Maureen <mau11@users.noreply.github.com>
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