Skip to content

Commit 6cd726d

Browse files
authored
configured stale action and minor updates to docs (#16)
1 parent bc43aad commit 6cd726d

File tree

5 files changed

+39
-17
lines changed

5 files changed

+39
-17
lines changed

.github/workflows/codecommitWorkflow.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
steps:
1313
- name: Check out code.
1414
uses: actions/checkout@v2
15-
- name: Run PySpelling as a GitHub action
15+
- name: Spelling check via PySpelling
1616
uses: sbates130272/spellcheck-github-actions@v0.6.0
1717
super-linter:
1818
# Job name
@@ -45,10 +45,10 @@ jobs:
4545
steps:
4646
- name: Check out code.
4747
uses: actions/checkout@v2
48-
- name: Validate JSON Schema
48+
- name: Validate src/mission.json against JSON Schema
4949
uses: docker://orrosenblatt/validate-json-action:latest
5050
env:
51-
INPUT_SCHEMA: /src/missions.schema.json
51+
INPUT_SCHEMA: ./src/missions.schema.json
5252
INPUT_JSONS: ./src/missions.json
5353
react-linting-and-testing:
5454
runs-on: ubuntu-latest
@@ -71,6 +71,7 @@ jobs:
7171
run: npm run coverage
7272
uml-validation:
7373
name: uml-validation
74+
needs: react-linting-and-testing
7475
runs-on: ubuntu-latest
7576
steps:
7677
- name: Check out code.

.github/workflows/pushtoMaster.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
name: Push to Master - Workflow
33
'on':
44
# Trigger this workflow on a push to the master branch
5+
# Work runs all linter checks, and tests if they are successful it builds and deploys application
56
push:
67
branches:
78
- master
@@ -12,7 +13,7 @@ jobs:
1213
steps:
1314
- name: Check out code.
1415
uses: actions/checkout@v2
15-
- name: Run PySpelling as a GitHub action
16+
- name: Spelling check via PySpelling
1617
uses: sbates130272/spellcheck-github-actions@v0.6.0
1718
super-linter:
1819
# Job name
@@ -45,7 +46,7 @@ jobs:
4546
steps:
4647
- name: Check out code.
4748
uses: actions/checkout@v2
48-
- name: Validate JSON Schema
49+
- name: Validate src/mission.json against JSON Schema
4950
uses: docker://orrosenblatt/validate-json-action:latest
5051
env:
5152
INPUT_SCHEMA: ./src/missions.schema.json
@@ -98,11 +99,12 @@ jobs:
9899
- name: Deploy Images
99100
shell: bash
100101
run: |
101-
echo "fetching the images from the images folder of the master branch into the images folder of the gh-pages branch"
102+
echo "Following deployment of the gh-pages branch, need to fetching the images from the images folder of the master branch "
103+
echo " into the images folder of the gh-pages branch. This is done to facilitate sequence to sequence links."
102104
git branch -a
103105
#git checkout gh-pages
104-
# checkout the images folder from the master branches
105-
# -- indicates do not interpret any more arguments as command options.
106+
# checkout the images folder from the master branch
107+
# the -- parameter indicates do not interpret any more arguments as command options.
106108
# and stage the files
107109
#git checkout master -- images
108110
git status
@@ -113,7 +115,7 @@ jobs:
113115
author_email: "tom_halpin@hotmail.com"
114116
message: "Deployed Images"
115117
add: "--all"
116-
#ref: "gh-pages"
118+
#ref: "gh-pages" # commit is to gh-pages branch
117119
env:
118120
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
119121

.github/workflows/stale.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,11 @@ jobs:
1818
- uses: actions/stale@v1
1919
with:
2020
repo-token: ${{ secrets.GITHUB_TOKEN }}
21-
stale-issue-message: 'Stale issue message'
22-
stale-pr-message: 'Stale pull request message'
21+
stale-issue-message: 'This issue has been marked as Stale because it has been open for 14 days with no activity. Remove the stale label or comment on the issue, or it will be closed in 7 days.'
22+
stale-pr-message: 'This PR has been marked as Stale because it has been open for 14 days with no activity. Remove the stale label or comment on the PR, or it will be closed in 7 days.'
2323
stale-issue-label: 'no-issue-activity'
24+
exempt-issue-labels: 'awaiting-approval,work-in-progress'
2425
stale-pr-label: 'no-pr-activity'
26+
exempt-pr-labels: 'awaiting-approval,work-in-progress'
27+
days-before-stale: 14
28+
days-before-close: 7

CONTRIBUTING.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,26 @@ We value and welcome all contributions including
1212

1313
## Contributing
1414

15+
### Sequences-as-Code repository
16+
17+
Contributions can be made to the improve the base repository.
18+
1519
* The way to collaborate and work on the project is via GitHub: [github.com/dxc-technology/Sequences-as-Code](https://github.com/dxc-technology/Sequences-as-Code)
1620
* For issues, bug fixes or adding new features, please create and submit a pull request
1721
* For raising an issue or a defect, please open a [new github issue](https://github.com/dxc-technology/Sequences-as-Code/issues/new)
1822

23+
### Your Missions and Sequences
24+
25+
You should not need to submit a pull request with your dedicated missions and sequences, i.e. missions specific to you and/or your organization.
26+
27+
For instructions on how to
28+
29+
* Stand up a dedicated instance of SaC
30+
* Add your missions and sequences
31+
* Perform common SaC tasks please
32+
33+
see the [documentation](./docs/README.md).
34+
1935
## Pull Requests
2036

2137
We encourage contributions from the community. No fix is too small.
@@ -32,3 +48,5 @@ If your pull request is not accepted at first, please try again after addressing
3248
* Submit **Pull Request** to the master repository including a brief description of the request. The **Pull Request** should include enough detail to enable the owner of the repository understand and process the request.
3349
* **@Mention** the team/team members in the comment section so the notification is sent. This alerts the team members of the need to review the code.
3450
* Wait for the repo admins to review the changes and provide **feedback**.
51+
52+
As noted above please do not submit a pull request with your dedicated missions and sequences, these should be added to a dedicated instance of SaC. For details on how to stand up a dedicated instance see the [documentation](./docs/README.md).

README.md

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# SaC (Sequences as Code) - README.md
22

3-
Repository for SaC (Sequences as Code)
3+
Repository for SaC (Sequences as Code).
44

55
## Purpose
66

@@ -44,14 +44,11 @@ We welcome all contributions please refer to [CONTRIBUTING.MD](/CONTRIBUTING.md)
4444

4545
## Issues
4646

47-
If you experience some problems with the existing scenarios, please open an
48-
[issue](https://github.com/dxc-technology/Sequences-as-Code/issues/new/choose)
49-
and let us know about them.
47+
If you experience some problems with the existing scenarios, please open an [issue](https://github.com/dxc-technology/Sequences-as-Code/issues/new/choose) and let us know about them.
5048

5149
## License
5250

53-
This project is licensed under the Mozilla Public License Version 2.0 - see
54-
the [LICENSE](LICENSE) file for details.
51+
This project is licensed under the Mozilla Public License Version 2.0 - see the [LICENSE](LICENSE) file for details.
5552

5653
## Create React App - Bootstrapped Readme.md
5754

0 commit comments

Comments
 (0)