Skip to content

Pilotage: Supprimer populate_metabase_flux_iae#5260

Merged
calummackervoy merged 1 commit into
masterfrom
calum/nettoyer-script-populate-fluxiae
Feb 17, 2025
Merged

Pilotage: Supprimer populate_metabase_flux_iae#5260
calummackervoy merged 1 commit into
masterfrom
calum/nettoyer-script-populate-fluxiae

Conversation

@calummackervoy

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Une fois que #5249 est mergé avec gip-inclusion/pilotage-airflow#368 le fonction de cette commande aura été remplacé par le DAG côté pilotage.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

@rsebille

Copy link
Copy Markdown
Contributor

@calummackervoy C'est peut-être déjà prévu mais maintenant que le DAG existe ça serais bien de fusionner cette PR rapidement pour éviter que Supportix et le DAG ne se marche dessus ou fasse la même chose 2 fois :).

No longer required since the data is now being managed by the Pilotage directly
@calummackervoy calummackervoy force-pushed the calum/nettoyer-script-populate-fluxiae branch from 49ad6dd to 7857387 Compare February 17, 2025 11:51
@calummackervoy calummackervoy marked this pull request as ready for review February 17, 2025 11:54
@calummackervoy

Copy link
Copy Markdown
Contributor Author

C'est prêt @rsebille :)

@rsebille rsebille left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nickel ! 😀

@calummackervoy calummackervoy added this pull request to the merge queue Feb 17, 2025
Merged via the queue into master with commit 65693d8 Feb 17, 2025
@calummackervoy calummackervoy deleted the calum/nettoyer-script-populate-fluxiae branch February 17, 2025 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants