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

Added Opened status for Sentinel PR #1355

Merged

Conversation

hermya
Copy link
Contributor

@hermya hermya commented Oct 21, 2024

Adding Opened status for time zone dependent test, with PR at: alibaba/Sentinel#3459

@darko-marinov
Copy link
Contributor

Please change the checker with the new category

@darko-marinov
Copy link
Contributor

Please rebase your changes on the latest main now that I accepted #1356

@darko-marinov
Copy link
Contributor

Not sure if you're working on The file is not properly ordered.

@hermya hermya force-pushed the fix-time-flaky-sentinel branch from 4c3ecf9 to a7c761c Compare October 22, 2024 03:56
@darko-marinov
Copy link
Contributor

Can you squash 8 commits into 1? You're just adding one new line, so no need for 8 commits to ad one line.

@hermya hermya force-pushed the fix-time-flaky-sentinel branch from a44c5b0 to eaf052a Compare October 22, 2024 04:18
@darko-marinov
Copy link
Contributor

This PR currently has some unnecessary changes in pr-data.csv (and two commits instead of one).

@hermya hermya force-pushed the fix-time-flaky-sentinel branch 2 times, most recently from b3491bc to 4c419d6 Compare October 22, 2024 04:29
@hermya hermya force-pushed the fix-time-flaky-sentinel branch from bea1578 to 991aa40 Compare October 22, 2024 04:35
@hermya
Copy link
Contributor Author

hermya commented Oct 22, 2024

Apologies for taking so many tries to reduce the changes to one commit

@darko-marinov darko-marinov merged commit 0d0a100 into TestingResearchIllinois:main Oct 22, 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.

3 participants