Skip to content

Commit

Permalink
Merge pull request #7 from thorwolpert/main
Browse files Browse the repository at this point in the history
adding issue & pr templates, and the ci scripts
  • Loading branch information
thorwolpert authored Mar 15, 2021
2 parents f1e7c52 + 604ff10 commit be46b13
Show file tree
Hide file tree
Showing 8 changed files with 185 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
# Default owner(s) for everything in the repo. Unless a later match takes
# precedence, will be requested for review when someone opens a pull request.

* @doug-lovett, @thorwolpert, @kialj876
35 changes: 35 additions & 0 deletions .github/ISSUE_TEMPLATE/bug_report.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
---
name: Bug report
about: Create a report to help us improve

---

**Describe the bug**
A clear and concise description of what the bug is.

**To Reproduce**
Steps to reproduce the behavior:
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

**Expected behavior**
A clear and concise description of what you expected to happen.

**Screenshots**
If applicable, add screenshots to help explain your problem.

**Desktop (please complete the following information):**
- OS: [e.g. iOS]
- Browser [e.g. chrome, safari]
- Version [e.g. 22]

**Smartphone (please complete the following information):**
- Device: [e.g. iPhone6]
- OS: [e.g. iOS8.1]
- Browser [e.g. stock browser, safari]
- Version [e.g. 22]

**Additional context**
Add any other context about the problem here.
17 changes: 17 additions & 0 deletions .github/ISSUE_TEMPLATE/feature.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
---
name: Feature
about: Describe a feature that will be estimated and considered for implementation

---

### Replace_With_Short_Descriptive_Name
## Description:


Acceptance / DoD:
- [ ] Design / Solution accepted by Product Owner
- [ ] Test coverage acceptable
- [ ] Peer Reviewed
- [ ] UX Approved
- [ ] PR Accepted
- [ ] Production burn in completed
17 changes: 17 additions & 0 deletions .github/ISSUE_TEMPLATE/feature_request.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
---
name: Feature request
about: Suggest an idea for this project

---

**Is your feature request related to a problem? Please describe.**
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

**Describe the solution you'd like**
A clear and concise description of what you want to happen.

**Describe alternatives you've considered**
A clear and concise description of any alternative solutions or features you've considered.

**Additional context**
Add any other context or screenshots about the feature request here.
16 changes: 16 additions & 0 deletions .github/ISSUE_TEMPLATE/task-or-ops.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
---
name: Task or Ops
about: Small tasks that are added to the sprints that requires a change to any repos

---

### Replace_With_Short_Descriptive_Name
## Description:


Acceptance / DoD:
- [ ] Product Owner advised if task >= 1 full-day, or forms part of the business day
- [ ] Requires deployments
- [ ] Test coverage acceptable
- [ ] Peer Reviewed
- [ ] Production burn in completed
6 changes: 6 additions & 0 deletions .github/PULL_REQUEST_TEMPLATE.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
*Issue #:* /bcgov/entity###

*Description of changes:*


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the registry-schemas license (Apache 2.0).
34 changes: 34 additions & 0 deletions .github/workflows/schema-pr.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
name: Schemas PR CI

on:
pull_request:
types: [assigned, synchronize]

jobs:
build:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: [3.7, 3.8]

steps:
- uses: actions/checkout@v2
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v1
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install -r requirements.txt
pip install -r requirements/dev.txt
- name: Lint with pylint
run: |
pylint --rcfile=setup.cfg --disable=C0301,W0511 --load-plugins=pylint_flask src/registry_schemas
- name: Lint with flake8
run: |
flake8 src tests
- name: Test with pytest
run: |
pip install .
pytest
56 changes: 56 additions & 0 deletions .github/workflows/schema-publish.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,56 @@
name: Schemas Publish Python Package

on:
push:
branches:
- master

jobs:
build:
runs-on: ubuntu-latest
strategy:
matrix:
python-version: [3.8]

steps:
- uses: actions/checkout@v2
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v1
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: |
python -m pip install --upgrade pip
pip install -r requirements.txt
pip install -r requirements/dev.txt
- name: Lint with pylint
run: |
pylint --rcfile=setup.cfg --disable=C0301,W0511 --load-plugins=pylint_flask src/registry_schemas
- name: Lint with flake8
run: |
flake8 src tests
- name: Test with pytest
run: |
pip install .
pytest
# publish:
# runs-on: ubuntu-latest
# needs: build
# steps:
# - uses: actions/checkout@v2
# - name: Set up Python
# uses: actions/setup-python@v1
# with:
# python-version: "3.x"
# - name: Install dependencies
# run: |
# python -m pip install --upgrade pip
# pip install setuptools wheel twine
# - name: Build and publish
# env:
# TWINE_USERNAME: ${{ secrets.ARTIFACTORY_SCHEMA_PYPI_USERNAME }}
# TWINE_PASSWORD: ${{ secrets.ARTIFACTORY_SCHEMA_PYPI_PASSWORD }}
# run: |
# python setup.py sdist bdist
# twine upload --repository-url ${{ secrets.ARTIFACTORY_SCHEMA_URL }} dist/*

0 comments on commit be46b13

Please sign in to comment.