Skip to content

2021csb1167/sof 98 youtube summaries page testing#134

Merged
Dhruv-Singh-Negi merged 15 commits intoArker123:mainfrom
Dhruv-Singh-Negi:2021csb1167/sof-98-youtube-summaries-page-testing
Mar 22, 2024
Merged

2021csb1167/sof 98 youtube summaries page testing#134
Dhruv-Singh-Negi merged 15 commits intoArker123:mainfrom
Dhruv-Singh-Negi:2021csb1167/sof-98-youtube-summaries-page-testing

Conversation

@Dhruv-Singh-Negi
Copy link
Collaborator

  1. Added Basic Tests For Summary Page Testing and Checking Api's Integration

@linear
Copy link

linear bot commented Mar 12, 2024

@Arker123
Copy link
Owner

The tests appear to be failing. Could you take a look into this? Thanks!

@Arker123
Copy link
Owner

As workflows aren't functioning at the moment, could you kindly execute these commands?

pip install pre-commit
pre-commit install --hook-type pre-commit
pre-commit run

@Dhruv-Singh-Negi
Copy link
Collaborator Author

C:\Users\dhruv\Desktop\T07-CS305>git commit -m "Increased Time limit For Hooks"
isort....................................................................Passed
black....................................................................Passed
client...................................................................Passed
server...................................................................Passed

Done, Please Review Again

…into 2021csb1167/sof-98-youtube-summaries-page-testing
@Dhruv-Singh-Negi Dhruv-Singh-Negi requested a review from n1ti March 22, 2024 17:04
Copy link
Owner

@Arker123 Arker123 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

@Dhruv-Singh-Negi Dhruv-Singh-Negi merged commit 4558ccc into Arker123:main Mar 22, 2024
@Dhruv-Singh-Negi Dhruv-Singh-Negi deleted the 2021csb1167/sof-98-youtube-summaries-page-testing branch March 22, 2024 17:16
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.

3 participants