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(networking): adding UDP to the roadmap #8

Merged
merged 2 commits into from May 17, 2024

Conversation

khoramism
Copy link
Contributor

This PR is associated with this PR in the data-engineering-wiki repo.

I've just added UDP to the subsection of Networking and I've also added Design Patterns section to the General Software Engineering skills.

thanks :D

@mostafaghadimi mostafaghadimi changed the title Feat: Adding Udp and Design Patterns section ⛽ feat(networking): adding UDP to the roadmap May 8, 2024
Copy link
Contributor

@mostafaghadimi mostafaghadimi left a comment

Choose a reason for hiding this comment

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

Hi @khoramism jan,

The edited file is not a valid YAML format and as you can see the pipeline is failed. Please just add UDP to the roadmap and we will discuss about the design patterns.

Thanks for your patience and contribution,
Mostafa

@khoramism
Copy link
Contributor Author

khoramism commented May 9, 2024

ofcourse, yeap my bad 🫣, I forgot to add some spaces, thanks for your patience!

@khoramism
Copy link
Contributor Author

I fixed it @mostafaghadimi jan

Copy link
Contributor

@mostafaghadimi mostafaghadimi left a comment

Choose a reason for hiding this comment

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

Well done @khoramism jan, LGTM.

@mostafaghadimi mostafaghadimi merged commit 694fd1e into data-burst:main May 17, 2024
1 check passed
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.

None yet

2 participants