-
Notifications
You must be signed in to change notification settings - Fork 369
[WIP] Workflow Update #210
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
Draft
Meltie2013
wants to merge
13
commits into
mangoszero:master
Choose a base branch
from
Prodigy-Project:workflow_update
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
+290
−55
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Updated workflow for code styling - Updated linux build file For the code styling, I have added 6 new checks that will check code for a few things. Refer to the PR description area for more details. Note: Making sure Linux is using the same version of OpenSSL as the Windows workflow build. So we are consistent with that package if problems happen. On the workflow side, default version is 3.0.2 from what I been seeing thus far.
2516c39 to
f4ffc33
Compare
- Working on some exception patterns that will output but will be marked with 0 instead of 1. Still a working progress. - Added general comments to the filters based on what they do.
- Added in some code to replace tab spaces with 4 spaces per tab Requested by @billy1arm
- Adding some paths ignore on some areas of the project, so only changes to the project source code will trigger a build and not areas like the ReadMe.md or apps folder.
- This script needs review based on the rules used in the script. Aggressive Level: Moderate-to-Strict
- This script should be only touching the src folder and the contents within it.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes Made:
New Tools;
Example:
Note:
With the Code Styling Check workflow, don't be surprised at what it spits out as I am still tweaking the regex out a bit more. The list is big just to say it.
More details will be added as I move along with the change/update.
This change is