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

moved application classes to their package to improve archetype gener… #112

Merged
merged 5 commits into from
Nov 3, 2021

Conversation

johnsall
Copy link
Contributor

@johnsall johnsall commented Nov 2, 2021

…ation

updated archetype config to improve generated project structure

📲 What

updated archetype config to improve generated project structure

🤔 Why

To improve archetype's generated project structure

🛠 How

👀 Evidence

Screenshots / external resources / links / etc.
Link to documentation updated with changes impacted in the PR

🕵️ How to test

Notes for QA

✅ Acceptance criteria Checklist

  • Code peer reviewed?
  • Documentation has been updated to reflect the changes?
  • Passing all automated tests, including a successful deployment?
  • Passing any exploratory testing?
  • Rebased/merged with latest changes from development and re-tested?
  • Meeting the Coding Standards?

@johnsall
Copy link
Contributor Author

johnsall commented Nov 2, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@johnsall
Copy link
Contributor Author

johnsall commented Nov 2, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@guillermogutierrez
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@guillermogutierrez
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 3, 2021

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@johnsall
Copy link
Contributor Author

johnsall commented Nov 3, 2021

lgtm (worked on it with @guillermogutierrez )

@johnsall johnsall merged commit df2d715 into main Nov 3, 2021
@guillermogutierrez guillermogutierrez deleted the fix/app-package branch November 3, 2021 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants