Skip to content

Workflow upgrade#90

Merged
carojeandat merged 13 commits intomainfrom
workflow-upgrade
Mar 17, 2026
Merged

Workflow upgrade#90
carojeandat merged 13 commits intomainfrom
workflow-upgrade

Conversation

@carojeandat
Copy link
Contributor

PR Summary

Update the build workflow to allow creating new applications that directly use the correct shared workflows.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

license-eye has totally checked 48 files.

Valid Invalid Ignored Fixed
23 1 24 0
Click to see the invalid file list
  • sonar-project.properties

@@ -0,0 +1,3 @@
sonar.projectKey=gridsuite_gridXXX-app
Copy link

Choose a reason for hiding this comment

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

Suggested change
sonar.projectKey=gridsuite_gridXXX-app
# This Source Code Form is subject to the terms of the Mozilla Public
# License, v. 2.0. If a copy of the MPL was not distributed with this
# file, You can obtain one at https://mozilla.org/MPL/2.0/.
sonar.projectKey=gridsuite_gridXXX-app

@antoinebhs
Copy link

Add other missing workflow. Just need to copy from existing apps

@khouadrired khouadrired self-requested a review March 17, 2026 09:04
@khouadrired
Copy link

the scan will always fail , as before we diddn't trigger the sonar scan for this template project.

carojeandat and others added 3 commits March 17, 2026 11:00
Co-authored-by: KHOUADRI Radouane <redouane.khouadri_externe@rte-france.com>
@sonarqubecloud
Copy link

Copy link

@khouadrired khouadrired left a comment

Choose a reason for hiding this comment

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

LGTM

@carojeandat carojeandat merged commit 7b8e445 into main Mar 17, 2026
5 checks passed
@carojeandat carojeandat deleted the workflow-upgrade branch March 17, 2026 10:29
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.

3 participants