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

🧹 Typo corrected "ou" --> "our" #923

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

AAYUSH-GUPTA-coder
Copy link

Made changes to multiple TOML files

Made changes to multiple TOML files
@AAYUSH-GUPTA-coder
Copy link
Author

Hi @janjakubnanista, I made changes to all 6 TOML files

@janjakubnanista janjakubnanista changed the title typo corrected "ou" --> "our" 🧹 Typo corrected "ou" --> "our" Oct 11, 2024
@janjakubnanista
Copy link
Contributor

Hey @AAYUSH-GUPTA-coder - your PR is failing because of an issue in our pipelines for PRs from forks. The fix is in #931 - once merged, your pipelines should pass, you'll just need to rebase

@ryandgoulding
Copy link
Contributor

Can you rebase @AAYUSH-GUPTA-coder ?

@janjakubnanista
Copy link
Contributor

@AAYUSH-GUPTA-coder you should now be able to rebase this branch on top of the newest main

@AAYUSH-GUPTA-coder
Copy link
Author

gm @janjakubnanista, sorry for the late reply. I had an accident and was hospitalized, which is why my response is delayed. Please check now.

@janjakubnanista
Copy link
Contributor

Hey @AAYUSH-GUPTA-coder! Sorry to hear that, hope you're good! Please rebase your branch - it seems to have 178 files changed at the moment, should be just 6 right?

Copy link
Contributor

@janjakubnanista janjakubnanista left a comment

Choose a reason for hiding this comment

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

Please rebase the branch to get rid of all the irrelevant changed files

@AAYUSH-GUPTA-coder
Copy link
Author

Sorry Ser, My Bad

Rebased now. Please check

@janjakubnanista

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