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

[Fix] Remove unecessary line #8295

Merged
merged 1 commit into from
Sep 30, 2024
Merged

[Fix] Remove unecessary line #8295

merged 1 commit into from
Sep 30, 2024

Conversation

adkif
Copy link
Contributor

@adkif adkif commented Sep 30, 2024

PR

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.


  • The setup window doesn't need to be within the time tracker's stop method.

@adkif adkif requested a review from evereq September 30, 2024 18:35
Copy link

nx-cloud bot commented Sep 30, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 19d202f. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


🟥 Failed Commands
nx serve api -c=production --prod

Sent with 💌 from NxCloud.

@evereq evereq changed the base branch from feat/use-window-manager-add-submenu to develop September 30, 2024 18:47
@evereq
Copy link
Member

evereq commented Sep 30, 2024

@adkif can you rebase please? I did something stupid when did not notice that you put PRs on top of each other branches, not develop etc

@evereq evereq merged commit b288004 into develop Sep 30, 2024
13 of 19 checks passed
@evereq evereq deleted the fix/remove-unecessary-line branch September 30, 2024 19:09
@evereq evereq restored the fix/remove-unecessary-line branch September 30, 2024 19:16
@evereq evereq deleted the fix/remove-unecessary-line branch September 30, 2024 19:17
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