Split up build_changeset in GitHub sync changeset modules into create_/update_

Offen

#1,193 erstellt am 14.11.2017 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 4/5
Geschätzter Aufwand: 3-5 Tage
Anfängerfreundlichkeit: 35/100

Issue-Typ: Refactoring
Klarheit: Muss geklärt werden
Aktivitätsstatus: Veraltet
Tech-Stack: elixir
Bereich: backend

Rechercherichtung

Untersuche zuerst die GitHub-Synchronisierungs-Changeset-Module und finde ihre build_changeset-Funktionen. Bilde das bestehende implizite Verhalten für Erstellen und Aktualisieren ab und überprüfe anschließend, dass separate create_- und update_-Changesets dieses Verhalten beibehalten und unabhängig voneinander per Unit-Test getestet werden können.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

needs estimate

Problem

Our build_changeset functions in some of our GitHub sync changeset modules are too implicit. We should break apart their behavior into more explicit create_ and update_ changesets. This has a side benefit of allowing for better unit testing of the changesets themselves.

Vorherrschende Sprache: Elixir
Sterne: 234
Forks: 82
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus code-corps/code-corps-api

URGENT: Security vulnerability detected in infrastructure config Offen

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 15/100

code-corps/code-corps-api#1385 · Erstellt am 06.1.2026
Cannot Create new GitHub App Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100

code-corps/code-corps-api#1384 · Erstellt am 12.12.2019
Track anon Users that register Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

code-corps/code-corps-api#1383 · 2 Kommentare · Erstellt am 12.3.2018
Use Dependabot to automatically keep dependencies up-to-date Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100

code-corps/code-corps-api#1382 · 2 Kommentare · Erstellt am 11.3.2018
Write tests for GitHub.Sync.pull_request_event/1 Offen

Difficulty: Medium Skill: Elixir Time: 4 hours

code-corps/code-corps-api#1372 · 3 Kommentare · 1 zugewiesene Person · Erstellt am 17.1.2018

Alle Issues in code-corps/code-corps-api