Skip to content

Conversation

@asafgabai
Copy link
Collaborator

@asafgabai asafgabai commented May 19, 2025

  • The pull request is targeting the main branch.
  • The code has been validated to compile successfully by running go vet ./....
  • The code has been formatted properly using go fmt ./....
  • All static analysis checks passed.
  • All tests have passed. If this feature is not already covered by the tests, new tests have been added.
  • All integration tests have passed locally as they cannot be automated yet.
  • All changes are detailed at the description. if not already covered at JFrog Documentation, new documentation have been added.

Added a new command for creating a new application.

jf app create [command options] --project=<value> <application-key>

@asafgabai asafgabai added new feature Automatically generated release notes safe to test Approve running end-to-end tests on a pull request labels May 19, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 19, 2025

Unit Tests Results

50 tests  +26   50 ✅ +26   0s ⏱️ ±0s
22 suites + 3    0 💤 ± 0 
 1 files   ± 0    0 ❌ ± 0 

Results for commit 0b7e0c3. ± Comparison against base commit 1645bf8.

♻️ This comment has been updated with latest results.

@asafgabai asafgabai marked this pull request as ready for review May 19, 2025 08:45
staticcheck:
# https://staticcheck.io/docs/options#checks
checks: [ "all","-SA1019","-SA1029" ]
gosec:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

G407 is not supported anymore so it can't be excluded. Also, exclude-test-files is not supported (and probably not wanted in the first place).

return nil, errorutils.CheckErrorf("--%s is mandatory", commands.ProjectFlag)
}

businessCriticality := ctx.GetStringFlagValue(commands.BusinessCriticalityFlag)
Copy link
Collaborator

Choose a reason for hiding this comment

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

consider extracting to util function, as it is the same exact usage for maturityLevel

@github-actions
Copy link
Contributor

👍 Frogbot scanned this pull request and did not find any new security issues.


@asafgabai asafgabai merged commit ed64b7d into main May 20, 2025
6 checks passed
@asafgabai asafgabai deleted the APP-440-Create-application-descriptor branch May 20, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new feature Automatically generated release notes safe to test Approve running end-to-end tests on a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants