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

Update development-updates.md Week15 #471

Closed
wants to merge 0 commits into from

Conversation

rose2221
Copy link
Contributor

No description provided.

@taxmeifyoucan
Copy link
Contributor

You don't need to open a new PR, just push updates to the right branch :)

@taxmeifyoucan
Copy link
Contributor

And reading the update now, it's pretty short so you could also merge it into a single document.

@rose2221
Copy link
Contributor Author

Should i explain the code that i wrote in the PR for TLS Protocol tests?

@taxmeifyoucan
Copy link
Contributor

Sure I believe you can elaborate on that! But the update should mainly serve you Rose, it's for you to collect your thoughts, write down what you learned and how your process works when working on this. So feel free to explain anything you find important, I don't really want to guide you here :)

@rose2221
Copy link
Contributor Author

The changes have been made. Thank you for your feedback :)

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