Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

return active colorscheme after colorscheme! #327

Merged
merged 1 commit into from
Jun 29, 2024

Conversation

MarcMush
Copy link
Contributor

@MarcMush MarcMush commented Jun 3, 2023

this was the case before #304

it breaks my startup.jl, where I modified the colorscheme using the returned value

@codecov-commenter
Copy link

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.03 🎉

Comparison is base (0d60955) 56.88% compared to head (fae9313) 56.92%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #327      +/-   ##
==========================================
+ Coverage   56.88%   56.92%   +0.03%     
==========================================
  Files          15       15              
  Lines        1111     1112       +1     
==========================================
+ Hits          632      633       +1     
  Misses        479      479              
Impacted Files Coverage Δ
src/OhMyREPL.jl 22.22% <100.00%> (+1.46%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@MarcMush
Copy link
Contributor Author

can this be merged?

@Moelf
Copy link
Contributor

Moelf commented Jun 14, 2023

I don't have commit rights sorry

@KristofferC KristofferC merged commit 2670a0a into KristofferC:master Jun 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants