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

[#2285] Split CI tasks #19

Closed
wants to merge 39 commits into from
Closed
Show file tree
Hide file tree
Changes from 32 commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
91af935
Update run script
CYX22222003 Jan 20, 2025
ad7b936
Update run.sh again
CYX22222003 Jan 20, 2025
bc94208
Add jar package
CYX22222003 Jan 20, 2025
9f115a5
Update run.sh
CYX22222003 Jan 20, 2025
ccfa314
Update ignore files
CYX22222003 Jan 20, 2025
9980b40
Add local website
CYX22222003 Jan 20, 2025
ae44039
Remove local generated files
CYX22222003 Jan 20, 2025
a73baed
Trigger rerun
CYX22222003 Jan 20, 2025
1d59cd0
Trigger rerun
CYX22222003 Jan 20, 2025
812315c
Trigger rerun
CYX22222003 Jan 20, 2025
ef156f4
Reset ignore
CYX22222003 Jan 20, 2025
274ea21
Remove jar
CYX22222003 Jan 20, 2025
c9291c7
Update run.sh
CYX22222003 Jan 20, 2025
a792a30
Update url
CYX22222003 Jan 20, 2025
28f4c4b
Experiment with url
CYX22222003 Jan 20, 2025
5e0881c
Update config
CYX22222003 Jan 20, 2025
9d3ac2c
Update config
CYX22222003 Jan 20, 2025
3b57133
Correct branch name
CYX22222003 Jan 20, 2025
3d09ea9
Update author config
CYX22222003 Jan 20, 2025
eb9510b
Update author config
CYX22222003 Jan 20, 2025
d22807e
Update bash again
CYX22222003 Jan 20, 2025
6f6f88d
Trigger auto rerun
CYX22222003 Jan 20, 2025
1381d8e
Update run script
CYX22222003 Jan 20, 2025
25b8dd8
Add netlify document
CYX22222003 Jan 31, 2025
7b4f2d3
Update error in build script
CYX22222003 Jan 31, 2025
1b35b38
Build personal website
CYX22222003 Jan 31, 2025
76c3ca9
Merge branch 'master' of https://github.com/CYX22222003/RepoSense-use…
CYX22222003 Jan 31, 2025
9af6ec9
Update CI tests
CYX22222003 Feb 8, 2025
e1eb0af
Merge branch 'master' of https://github.com/reposense/publish-RepoSen…
CYX22222003 Feb 8, 2025
19d81bb
Add test scripts
CYX22222003 Feb 8, 2025
b9ce4e4
Remove changes to deploy personal dashboard
CYX22222003 Feb 8, 2025
b3d6f07
Rename job name
CYX22222003 Feb 8, 2025
3f03c20
Combine ci jobs into one file
CYX22222003 Feb 9, 2025
dbe6bf2
Remove redundant files
CYX22222003 Feb 9, 2025
fccb5d9
Add dependencies to jobs
CYX22222003 Feb 9, 2025
9eb2c50
Add upload artifact
CYX22222003 Feb 9, 2025
f108bff
Restrict deployment to push only
CYX22222003 Feb 9, 2025
da8bd93
Update gitignore and main.yml
CYX22222003 Feb 9, 2025
94cb7e5
Remove redundant tests
CYX22222003 Feb 9, 2025
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
2 changes: 2 additions & 0 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ jobs:
export VITE_BASE_DIR="/$(basename $GITHUB_REPOSITORY)/"
echo "VITE_BASE_DIR: $VITE_BASE_DIR"
./run.sh
- name: Predeployment test
run: bash ./test.sh
- name: Deploy GitHub pages
uses: peaceiris/actions-gh-pages@v3
with:
Expand Down
21 changes: 21 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
name: test

on:
pull_request:


jobs:
test:
runs-on: ubuntu-22.04

steps:
- uses: actions/checkout@v2
- name: Install dependencies
run: pip install requests
- name: Generate report
run: |
export VITE_BASE_DIR="/$(basename $GITHUB_REPOSITORY)/"
echo "VITE_BASE_DIR: $VITE_BASE_DIR"
./run.sh
- name: Predeployment test
run: bash ./test.sh
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
reposense-report/
RepoSense/
reposense/
test/
1 change: 1 addition & 0 deletions config/author-config.csv
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
Repository's Location,Branch,Author's Git Host ID, Author's Emails, Author's Display Name,Author's Git Author Name,Ignore Glob List
,,,,,,
2 changes: 2 additions & 0 deletions config/group-config.csv
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
Repository's Location,Group Name,Globs
,,
5 changes: 5 additions & 0 deletions test.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
if ! ls ./reposense-report | grep -q "^index.html$"; then
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feels a little bit superficial, and with extensive tests on the main reposense project, doesn't seem entirely necessary.

If there isn't something substantial here that isn't subsumed by the main reposense tests, let's remove this for now.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is my attempt to address the requirement of automating the testing of deployments mentioned in this issue. I am not sure whether the "testing" mentioned in the issue is referring to a pre-deployment or post-deployment testing.

Copy link
Contributor

@gok99 gok99 Feb 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it refers to being able to test for things like empty reports in the deployed site (when it should not be).

However, the test here wouldn't be able to check for this, and it wouldn't be visible from the main repository. Let's add testing incrementally, and see if there something substantial to test in a separate PR.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, got it!

echo "index.html not found!"
exit 1
fi
echo "All tests passed."