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.

Sign up for GitHub

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

Jump to bottom

Remove HTTPS from task_4 #1715

Merged
merged 1 commit into from
Oct 21, 2024
Merged

Remove HTTPS from task_4 #1715

merged 1 commit into from
Oct 21, 2024

Conversation

kustikov112
Copy link
Contributor

task_4 hot fix

in order to simplify the task - HTTPS implementation is removed from task4

🤔 This is a ...

  • 🌟 New task
  • 🌐 New module
  • [ x ] ⚙️ Update to an existing task
  • 🔧 Update to an existing module
  • 🔗 Update or addition of external resources or links
  • [ x ] 🐛 Fix in a task or related content
  • 🛠 Fix in a module or related content
  • ✏️ Fixed a typo or grammatical error
  • 🔗 Fixed a broken link
  • ❓ Other (specify: ****____****)

Description

Points have been rearranged a bit. HTTPS implementation have been removed from the task.

Additional Information

  • Screenshots/Links:
  • Related Issues:

Checklist

  • [x ] ✅ I have performed a self-review of my own code.
  • 📝 I have commented my code, particularly in hard-to-understand areas.
  • 🔧 I have made corresponding changes to the documentation (if applicable).
  • [x ] 🚫 My changes generate no new warnings or errors.

@REW1L
Copy link
Collaborator

REW1L commented Oct 21, 2024

LGTM! Thank you @kustikov112!

@REW1L REW1L merged commit 13cf54d into rolling-scopes-school:master Oct 21, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kustikov112 @REW1L