Skip to content

Commit 3270958

Browse files
Merge pull request #291 from asgardeo/pavinduLakshan-patch-5
Remove redundant steps in release workflow
2 parents 24d3bca + ce22177 commit 3270958

File tree

2 files changed

+7
-73
lines changed

2 files changed

+7
-73
lines changed

.github/workflows/release.yml

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -80,32 +80,3 @@ jobs:
8080
env:
8181
GITHUB_TOKEN: ${{ secrets.ASGARDIO_GITHUB_BOT_TOKEN }}
8282
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
83-
84-
- name: Pack Release Artifacts
85-
if: steps.changesets.outputs.hasChangesets == 'false'
86-
run: |
87-
mkdir artifacts
88-
if test -d samples
89-
then
90-
cd samples
91-
rm -rf **/node_modules/
92-
rm -rf samples/**/dist/
93-
rm -rf samples/asgardeo-choreo-react-express/apps/**/node_modules/
94-
rm -rf samples/asgardeo-choreo-react-express/apps/client/build/
95-
rm -rf samples/asgardeo-choreo-react-express/apps/server/dist/
96-
for dir in */ ; do
97-
zip -r "../artifacts/${dir%/}.zip" $dir
98-
done
99-
fi
100-
101-
- name: Release Asgardeo Auth React SDK
102-
if: steps.changesets.outputs.hasChangesets == 'false'
103-
run: |
104-
version=`git describe --tags --abbrev=0`
105-
echo ${{secrets.ASGARDIO_GITHUB_BOT_TOKEN}} | gh auth login --with-token
106-
if test -d samples
107-
then
108-
gh release create $version artifacts/*
109-
else
110-
gh release create $version
111-
fi

pull_request_template.md

Lines changed: 7 additions & 44 deletions
Original file line numberDiff line numberDiff line change
@@ -1,52 +1,15 @@
11
## Purpose
2-
> Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.
2+
<!-- Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc. -->
33

4-
## Goals
5-
> Describe the solutions that this feature/fix will introduce to resolve the problems described above
6-
7-
## Approach
8-
> Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email documentation@wso2.com to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.
9-
10-
## User stories
11-
> Summary of user stories addressed by this change>
12-
13-
## Release note
14-
> Brief description of the new feature or bug fix as it will appear in the release notes
15-
16-
## Documentation
17-
> Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact
18-
19-
## Training
20-
> Link to the PR for changes to the training content in https://github.com/wso2/WSO2-Training, if applicable
21-
22-
## Certification
23-
> Type “Sent” when you have provided new/updated certification questions, plus four answers for each question (correct answer highlighted in bold), based on this change. Certification questions/answers should be sent to certification@wso2.com and NOT pasted in this PR. If there is no impact on certification exams, type “N/A” and explain why.
24-
25-
## Marketing
26-
> Link to drafts of marketing content that will describe and promote this feature, including product page changes, technical articles, blog posts, videos, etc., if applicable
27-
28-
## Automation tests
29-
- Unit tests
30-
> Code coverage information
31-
- Integration tests
32-
> Details about the test cases and coverage
4+
## Related issues
335

346
## Security checks
35-
- Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
36-
- Ran FindSecurityBugs plugin and verified report? yes/no
37-
- Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no
7+
- [] Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines
8+
- [] Ran FindSecurityBugs plugin and verified report
9+
- [] Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets
3810

3911
## Samples
40-
> Provide high-level details about the samples related to this feature
12+
<!-- Provide high-level details about the samples related to this feature -->
4113

4214
## Related PRs
43-
> List any other related PRs
44-
45-
## Migrations (if applicable)
46-
> Describe migration steps and platforms on which migration has been tested
47-
48-
## Test environment
49-
> List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested
50-
51-
## Learning
52-
> Describe the research phase and any blog posts, patterns, libraries, or add-ons you used to solve the problem.
15+
<!-- List any other related PRs -->

0 commit comments

Comments
 (0)