Skip to content

280 implement pipeline builder for section#362

Merged
mikehquan19 merged 8 commits intodevelopfrom
280-implement-pipeline-builder-for-section
Apr 13, 2026
Merged

280 implement pipeline builder for section#362
mikehquan19 merged 8 commits intodevelopfrom
280-implement-pipeline-builder-for-section

Conversation

@mikehquan19
Copy link
Copy Markdown
Contributor

@mikehquan19 mikehquan19 requested a review from ruba0s March 20, 2026 04:35
@mikehquan19
Copy link
Copy Markdown
Contributor Author

mikehquan19 commented Apr 1, 2026

@Krish-Patel656 Your pipeline works well, but can you reference to the pipeline builder we got incourse.go and professor.go and rewrite your builder based on them? I just want a bit of consistency.

You don't have to replicate their code. Just use their structure and apply them to your function.

@Krish-Patel656
Copy link
Copy Markdown
Contributor

Krish-Patel656 commented Apr 3, 2026

@mikehquan19 I tried to replicate the structure of the other files if you would like to take a look.

@mikehquan19
Copy link
Copy Markdown
Contributor Author

@Krish-Patel656 I will take a look this weekend :)

@mikehquan19
Copy link
Copy Markdown
Contributor Author

It looks good now!

@mikehquan19 mikehquan19 merged commit 547bd81 into develop Apr 13, 2026
2 checks passed
@mikehquan19 mikehquan19 deleted the 280-implement-pipeline-builder-for-section branch April 13, 2026 00:17
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