-
Notifications
You must be signed in to change notification settings - Fork 0
Add test results summary and group logs #1
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
Open
mw-kapilg
wants to merge
22
commits into
main
Choose a base branch
from
kapilg/refactor
base: main
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.
Open
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
fcf2354
set up initial utilities
mw-kapilg 7b0e28f
update package.json
mw-kapilg b0fb225
add index.ts
mw-kapilg c43caf6
add devDeps
mw-kapilg 352fc42
add setup script
mw-kapilg 3f68d47
add prepare script
mw-kapilg 1fb6ac6
export scripts and plugins
mw-kapilg 4e9cba1
update plugins path
mw-kapilg 1db39f9
set up for other action repos and update tests
mw-kapilg 9ca2510
add github workflows
mw-kapilg a33b4fc
add run-command and run-build modules
mw-kapilg 3d36b7f
update publish.yml
mw-kapilg dc7428e
update artifact name
mw-kapilg 27216e6
update publish.yml
mw-kapilg bb61cea
debug artifact presence
mw-kapilg 1fc32c9
use core.info
mw-kapilg 34ec70b
update artifact name
mw-kapilg 62a862e
fix test failure
mw-kapilg 07ac304
restructure plugins
mw-kapilg 0f72e88
update view generation preferences
mw-kapilg a0a19f7
update matlab release in getDefaultPlugins
mw-kapilg 72328c9
update as per feedback
mw-kapilg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,2 @@ | ||
| dist/* binary | ||
| lib/* binary |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| name: Build and Test | ||
| on: [push] | ||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| bat: | ||
| name: Build and Test | ||
| runs-on: ubuntu-22.04 | ||
| steps: | ||
| - uses: actions/checkout@v5 | ||
| - uses: actions/setup-node@v5 | ||
| with: | ||
| node-version: 20 | ||
| - name: Perform npm tasks | ||
| run: npm run ci |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,77 @@ | ||
| name: Publish | ||
| on: | ||
| release: | ||
| types: published | ||
| permissions: | ||
| contents: write | ||
|
|
||
| jobs: | ||
| build: | ||
| name: Build | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| tag: ${{ steps.update-package-version.outputs.version }} | ||
| steps: | ||
| - uses: actions/checkout@v5 | ||
| - name: Configure git | ||
| run: | | ||
| git config user.name 'Release Action' | ||
| git config user.email '<>' | ||
| - uses: actions/setup-node@v5 | ||
| with: | ||
| node-version: 20 | ||
|
|
||
| # Call `npm version`. It increments the version and commits the changes. | ||
| # We'll save the output (new version string) for use in the following | ||
| # steps | ||
| - name: Update package version | ||
| id: update-package-version | ||
| run: | | ||
| git tag -d "${{ github.event.release.tag_name }}" | ||
| VERSION=$(npm version "${{ github.event.release.tag_name }}" --no-git-tag-version) | ||
| git add package.json package-lock.json | ||
| git commit -m "[skip ci] Bump $VERSION" | ||
| git push origin HEAD:main | ||
|
|
||
| # Now carry on, business as usual | ||
| - name: Perform npm tasks | ||
| run: npm run ci | ||
|
|
||
| # Finally, create a detached commit containing the built artifacts and tag | ||
| # it with the release. Note: the fact that the branch is locally updated | ||
| # will not be relayed (pushed) to origin | ||
| - name: Commit to release branch | ||
| id: release_info | ||
| run: | | ||
| # Check for semantic versioning | ||
| longVersion="${{github.event.release.tag_name}}" | ||
| echo "Preparing release for version $longVersion" | ||
| [[ $longVersion == v[0-9]*.[0-9]*.[0-9]* ]] || (echo "must follow semantic versioning" && exit 1) | ||
| majorVersion=$(echo ${longVersion%.*.*}) | ||
| minorVersion=$(echo ${longVersion%.*}) | ||
|
|
||
| # Add the built artifacts. Using --force because dist/lib should be in | ||
| # .gitignore | ||
| git add --force dist lib | ||
|
|
||
| # Make the commit | ||
| MESSAGE="Build for $(git rev-parse --short HEAD)" | ||
| git commit --allow-empty -m "$MESSAGE" | ||
| git tag -f -a -m "Release $longVersion" $longVersion | ||
|
|
||
| # Get the commit of the tag you just released | ||
| commitHash=$(git rev-list -n 1 $longVersion) | ||
|
|
||
| # Delete the old major and minor version tags locally | ||
| git tag -d $majorVersion || true | ||
| git tag -d $minorVersion || true | ||
|
|
||
| # Make new major and minor version tags locally that point to the commit you got from the "git rev-list" above | ||
| git tag -f $majorVersion $commitHash | ||
| git tag -f $minorVersion $commitHash | ||
|
|
||
| # Force push the new minor version tag to overwrite the old tag remotely | ||
| echo "Pushing new tags" | ||
| git push -f origin $longVersion | ||
| git push -f origin $majorVersion | ||
| git push -f origin $minorVersion |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,123 @@ | ||
| ## For development environment | ||
| .vscode | ||
|
|
||
| # Don't include intermediate TypeScript compilation; | ||
| # it should be forcibly added on release | ||
| lib | ||
|
|
||
| # Leave out dist; it should be forcibly added on release | ||
| dist | ||
|
|
||
| ## https://raw.githubusercontent.com/github/gitignore/master/Node.gitignore | ||
| # Logs | ||
| logs/ | ||
| *.log* | ||
|
|
||
| # Diagnostic reports (https://nodejs.org/api/report.html) | ||
| report.[0-9]*.[0-9]*.[0-9]*.[0-9]*.json | ||
|
|
||
| # Runtime data | ||
| pids | ||
| *.pid | ||
| *.seed | ||
| *.pid.lock | ||
|
|
||
| # Directory for instrumented libs generated by jscoverage/JSCover | ||
| lib-cov | ||
|
|
||
| # Coverage directory used by tools like istanbul | ||
| coverage | ||
| *.lcov | ||
|
|
||
| # nyc test coverage | ||
| .nyc_output | ||
|
|
||
| # Grunt intermediate storage (https://gruntjs.com/creating-plugins#storing-task-files) | ||
| .grunt | ||
|
|
||
| # Bower dependency directory (https://bower.io/) | ||
| bower_components | ||
|
|
||
| # node-waf configuration | ||
| .lock-wscript | ||
|
|
||
| # Compiled binary addons (https://nodejs.org/api/addons.html) | ||
| build/Release | ||
|
|
||
| # Dependency directories | ||
| node_modules/ | ||
| jspm_packages/ | ||
|
|
||
| # Snowpack dependency directory (https://snowpack.dev/) | ||
| web_modules/ | ||
|
|
||
| # TypeScript cache | ||
| *.tsbuildinfo | ||
|
|
||
| # Optional npm cache directory | ||
| .npm | ||
|
|
||
| # Optional eslint cache | ||
| .eslintcache | ||
|
|
||
| # Microbundle cache | ||
| .rpt2_cache/ | ||
| .rts2_cache_cjs/ | ||
| .rts2_cache_es/ | ||
| .rts2_cache_umd/ | ||
|
|
||
| # Optional REPL history | ||
| .node_repl_history | ||
|
|
||
| # Output of 'npm pack' | ||
| *.tgz | ||
|
|
||
| # Yarn Integrity file | ||
| .yarn-integrity | ||
|
|
||
| # dotenv environment variables file | ||
| .env | ||
| .env.test | ||
|
|
||
| # parcel-bundler cache (https://parceljs.org/) | ||
| .cache | ||
| .parcel-cache | ||
|
|
||
| # Next.js build output | ||
| .next | ||
| out | ||
|
|
||
| # Nuxt.js build / generate output | ||
| .nuxt | ||
| # dist | ||
|
|
||
| # Gatsby files | ||
| .cache/ | ||
| # Comment in the public line in if your project uses Gatsby and not Next.js | ||
| # https://nextjs.org/blog/next-9-1#public-directory-support | ||
| # public | ||
|
|
||
| # vuepress build output | ||
| .vuepress/dist | ||
|
|
||
| # Serverless directories | ||
| .serverless/ | ||
|
|
||
| # FuseBox cache | ||
| .fusebox/ | ||
|
|
||
| # DynamoDB Local files | ||
| .dynamodb/ | ||
|
|
||
| # TernJS port file | ||
| .tern-port | ||
|
|
||
| # Stores VSCode versions used for testing VSCode extensions | ||
| .vscode-test | ||
|
|
||
| # yarn v2 | ||
| .yarn/cache | ||
| .yarn/unplugged | ||
| .yarn/build-state.yml | ||
| .yarn/install-state.gz | ||
| .pnp.* |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,14 @@ | ||
| ## No need to work on final dist | ||
| dist | ||
|
|
||
| ## Ignore external node_modules | ||
| node_modules | ||
|
|
||
| ## Don't include intermediate TypeScript compilation | ||
| tsout | ||
|
|
||
| ## Ignore code coverage | ||
| coverage | ||
|
|
||
| ## Editor related | ||
| .vscode |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| tabWidth: 4 | ||
| printWidth: 100 | ||
| overrides: | ||
| - files: "*.yml" | ||
| options: | ||
| tabWidth: 2 |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| # common-utils | ||
| # common-utils | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| # Reporting Security Vulnerabilities | ||
|
|
||
| If you believe you have discovered a security vulnerability, please report it to | ||
| [security@mathworks.com](mailto:security@mathworks.com). Please see | ||
| [MathWorks Vulnerability Disclosure Policy for Security Researchers](https://www.mathworks.com/company/aboutus/policies_statements/vulnerability-disclosure-policy.html) | ||
| for additional information. |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| module.exports = { | ||
| preset: "ts-jest", | ||
| testEnvironment: "node", | ||
| testRunner: "jest-circus/runner", | ||
| collectCoverage: true, | ||
| }; |
File renamed without changes.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will this repo be public? Do you plan on adding some information to the README?
(A brief README example: https://github.com/matlab-actions/workflow-generator)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another example: https://github.com/mathworks/matlab-azure-devops-extension/blob/master/README.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It will have to public since the rest of the project is open source. A README would probably be beneficial, although I don't see it as a super critical priority since the repo isn't intended for direct customer use (and that should be clear via context)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I don't mind deleting the README in its current form. If we want to keep it in the repo, then perhaps a sentence or two would be necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The repo will indeed need to be public. I'm not sure what are the guidelines for a public repo.
I'm okay with removing the README as well if the guidelines don't have a strict requirement. Yeah, makes sense to add a few lines in the README if we need to keep it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mcafaro, do you think having a README is a requirement for this repo?