#27 merge .githubs #30
Conversation
|
Do we know why the CI tests are failing/ can they be fixed? @zmorris248 @mbrzus we need to update the ruleset so that PR's failing CI cannot be merged into main. |
@Ivan-E-Johnson @zmorris248 looks like some jobs are failing because files can't be found on certain paths should be easy fixes hopefully |
|
@Ivan-E-Johnson @zmorris248 I think I have pushed fixes for correcting the file paths however now 3/4 errors we get might be due to permissions? I am not sure The error in the frontend CI execute lint happens with the setup-node action and is identical to the one discussed in actions/setup-node#624 I attempted what they outlined but didn't have any luck. Can someone else try? |
|
@powersaudrey25 sounds good, thank you for investigating further. I think we need to comment the invalid CI/workflow sections out. IE revert the workflow to a state where CI passed on valid changes and fails on invalid. We shouldn't fail on valid changes due to incomplete setup. |
|
As for the rulesets. I disabled the ruleset for the main branch due to allow the merge of the repos. As this is already completed, I will enable the ruleset back on work on modifying it to include the requirement to pass the github action. If I understand correctly, the CI workflow are for the automated AWS deployment, correct? @Ivan-E-Johnson:
|
|
I have been looking into the failing tests, I believe that the backend, trackerapi, and cdk infra fails may be due to permissions. I am not sure, but it looks like the tests are trying to modify the coverage.xml file in those folders with a comment after running the tests, but we do not currently give github actions permission to modify files in our repo. Changing this might resolve these issues. If we keep encountering issues, i agree with @Ivan-E-Johnson 's approach. |
|
I upgraded the node version and downgraded a dependency for the frontend and now the frontend ci fully passes. All that is left to fix are those permissions for backend, cdk, trackerapi. |
Ivan-E-Johnson
force-pushed
the
#27-merge-.githubs
branch
from
f1cc2da to
f6e7734
Compare
Ivan-E-Johnson
added
bug
|
@mbrzus can you review this and approve before merging? Also Im concerned that it is asking me to rebase and merge with out needing a PR |
I am moving the backend workflow file to our root .github folder
Moving cdk infra workflow to root workflow folder
After moving all workflow and necessary files to root .github directory, I am removing this unnecessary .github directory
This is a work arround until we get write permissions and can add github level coverage reporting to the repo
mbrzus
force-pushed
the
#27-merge-.githubs
branch
from
f6e7734 to
302b642
Compare
|
The previous Pull Request changed the requirements file path in the dockerfile. This is a potential cause for the checks to fail. There was also a .gitignore conflict, bt this was resolved and I do not anticipate that to be causing the failures. |
We may want to do a cleaning of these requirements files
Overview
We are integrating the old repo's .github folder into our own.
Implementation
I have moved all of the workflow files from the job-monitoring-app to our root .github folder in a new workflows folder.
I have deleted the non-root .github folder in job-monitoring-app