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

add workflow for syncing #76

Closed
wants to merge 4 commits into from
Closed
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions .github/workflows/sync_branch.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
name: sync main with dev
on:
push:
branches:
- main

jobs:
sync-branches:
runs-on: ubuntu-latest
name: syncing main with dev
steps:
- name: Checkout
uses: actions/checkout@v2
- name: Set up Node
uses: actions/setup-node@v1
with:
node-version: 12
- name: Opening pull request
id: pull
uses: tretuna/[email protected]
with:
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
FROM_BRANCH: "main"
TO_BRANCH: "dev"
6 changes: 5 additions & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,18 @@ Thanks for the interest in contributing to NeMo-Aligner. We do all of NeMo-Align

# Pull Requests (PR) Guidelines

**Send your PRs to the `main` branch**
**Send your PRs to the `main` or `dev` branch**

1) Make sure your PR does one thing. Have a clear answer to "What does this PR do?".
2) Read General Principles and style guide below
3) Make sure you sign your commits. E.g. use ``git commit -sS`` when committing.
4) Make sure all unittests finish successfully before sending PR ``pytest`` or (if your dev box does not have GPU) ``pytest --cpu`` from the root folder
5) Send your PR and request a review

**NOTE**: The `main` branch uses a fixed nemo version which we will update on every release. The `dev` branch is the branch that has all the commits from aligner main but is up to date with nemo main, this branch is less stable but we run nightly tests on this branch to make sure everything works. We only provide the dockerfile that works with `main`.
gshennvm marked this conversation as resolved.
Show resolved Hide resolved

Every release `dev` and `main` will sync to be the same.

## Unit tests
Quick unit tests (locally, while developing)
```
Expand Down
Loading