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

feat(appunti): Caricati vari file pdf #61

Closed
wants to merge 3 commits into from
Closed

feat(appunti): Caricati vari file pdf #61

wants to merge 3 commits into from

Conversation

Gaiasem
Copy link
Contributor

@Gaiasem Gaiasem commented Oct 3, 2023

No description provided.

@ncvescera ncvescera changed the title Add files via upload feat(appunti): Caricati vari file pdf Oct 4, 2023
@ncvescera
Copy link
Member

@Gaiasem Grazie mille per gli appunti, da una prima occhiata sembrano molto interessanti 🌚

Dovresti però fare come hai fatto per la pull request #60: una tipologia di appunti per PR.
Ti chiedo quindi di creare un branch (a partire dal tuo master, quello con tutti e 3 i nuovi file) per ogni nuovo file, e ognuno di questi deve contenere solo 1 nuovo file (tranne che per linguaggi che ha 2 fie quindi li metti tutti e due nel branch relativo a inguaggi). Poi aprirai 3 pull request diverse, una per ogni nuovo branch creato.
In poche parole quindi:

  • Crea 3 nuovi branch:
    • linguaggi: in questo branch ci metti solo i due file di linguaggi
    • ingegneria: in questo solo il file di ingengeria del software
    • architettura-reti: in questo solo qello di architettura
  • Apri una PR per ogni branch:
    • PR per linguaggi usando il branch linguaggi
    • PR per ingegneria del software usando il branch ingegneria
    • PR per architettura reti usando il branch architettura-reti

Aspetto che vedi il commento e poi chiudo questa PR 🤖

@ncvescera ncvescera self-assigned this Oct 4, 2023
@Gaiasem Gaiasem closed this Oct 4, 2023
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