Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

merge inhouse and non inhouse pipelines

Aperta
  • Qualcosa non va?
#22 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
38/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
github-actions
Ambito
ci-cd, infrastructure

Direzione di ricerca

Confronta .github/workflows/create-kustomize-with-bratiska-cli-inhouse.yml con .github/workflows/create-kustomize-with-bratiska-cli.yml per identificare la logica della pipeline duplicata e la configurazione attuale del runner. Consolida i workflow in un unico file con un meccanismo chiaro di selezione del runner, quindi verifica che i percorsi di esecuzione GitHub e in-house siano ancora disponibili e che il workflow ridondante sia stato rimosso.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

enhancement

'm starting to think that splitting this into two files wasn't the best idea. We are just copying the same code twice. Which goes directly against DRY principle.

My suggestion would be to keep just one file - .github/workflows/create-kustomize-with-bratiska-cli-inhouse.yml and drop "-inhouse" from the name. Then if you would like to run it on GitHub runners, you can just specify

runs-on: ['ubuntu-latest']
and it does the same thing and it is almost as many characters as specify "-bratiska-cli.yml" or "-bratiska-cli-inhouse.yml". Or, if you would like to be even more user friendly, you can just rework this into:

  runs-on:
    description: "Define environment where the job will run. Possible values: ['github', 'inhouse']"
    type: choice
    options: 
      - github
      - inhouse
    required: false
    default: 'inhouse'

Then you can just have a basic step that makes that decision inside a pipeline

  - name: Pick a runner
    run: |
      if [[ "${inputs.runs-on}" == "github" ]]; then
          # print and set env variable
      elif [[ "${inputs.runs-on}" == "inhouse" ]]; then
          # print and set env variable
      else
          # throw an error about unknown runner environment
      fi

This leaves the door open for multiple environments in the future and is still as user friendly as possible and you can easily switch just by specifying different input.

I'm flagging this here, as .github/workflows/create-kustomize-with-bratiska-cli-inhouse.yml and .github/workflows/create-kustomize-with-bratiska-cli.yml are new files and we can easily change it here. We don't have to make this change now for all of the files and all of the repositories. That can be pushed to latter PRs/changes.

Lingua principale
Nessun dato sulla lingua
Stelle
0
Fork
0
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di bratislava/github-actions

Tutte le issue di bratislava/github-actions

Issue simili

Altre issue su DevOps

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.