forked from dandi/dandi-infrastructure
-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Description
Steps
- Create new branch
sync-...fromapl-setup - Sync
masterwith (from)dandi/dandi-infrastructure:master- Using GitHub buttton on this repository's home page
- Open a pull request to merge
masterintosync-...mastercontains all of the dandi/dandi-infrastructure update- while
sync-...contains all of our EMBER updates
- Review changes
- Resolve merge conflicts
- I usually do this locally by merging
masterintosync-...and dealing with conflicts in VSCode. Because, we don't want to add any of our EMBER changes to themasterbranch. We wantmasterto always be "dandi-only" code
- I usually do this locally by merging
- PR closed
Now, sync-... contains all of the updates both from dandi-instructure and our fork!
- Open pull request to merge
sync-...intoapl-setupto update our infrastructure (sandbox + prod) in terraform - Extra Step for this Issue: Merge Set environment vars to set vendor-specific dandi-schema used by the DANDI instance #47 into
sync-...branch.- this MR contains "EMBER-ifying" changes related to the changes from dandi
- Review changes
- Merge PR
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels