-
Notifications
You must be signed in to change notification settings - Fork 448
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
extensions/kde-neon-6: fix XDG_CONFIG_DIRS #5104
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5104 +/- ##
==========================================
- Coverage 94.88% 89.70% -5.18%
==========================================
Files 658 341 -317
Lines 55189 22587 -32602
==========================================
- Hits 52364 20262 -32102
+ Misses 2825 2325 -500 ☔ View full report in Codecov by Sentry. |
Thank you, except this PR needs to be done here: https://invent.kde.org/neon/snap-packaging/snapcraft-desktop-integration |
Let's merge it here until it's used from here, else we'll not be able to use the change. |
e119025
to
b900513
Compare
@sergiusens @lengau I think we can merge this. The Code Coverage issue is not related to this I think. |
@soumyaDghosh can you please also do a pr here for this https://invent.kde.org/neon/snap-packaging/snapcraft-desktop-integration |
Very sorry, I forgot about this. It's done! https://invent.kde.org/neon/snap-packaging/snapcraft-desktop-integration/-/merge_requests/2 |
@sergiusens @lengau kindly merge this... |
I'm running into this and it's very frustrating. Is there anything I can do to help getting this merged? @soumyaDghosh |
This MR should be merged. If you want a quick work around for existing snaps, then create a symlink of .config/kdeglobals in |
Adding the symlink didn't help, thanks anyway though! I'll have to be patient. |
Which snap is it? Also what's the exact path did you put the file? |
This is for kasts. I ran '~/snap/kasts/current/.config$ ln --symbolic ~/.config/kdeglobals' |
So is there, in |
Yes!On Dec 6, 2024, at 22:14, সৌম্যদীপ ঘোষ ***@***.***> wrote:
So is there, in snap/kasts/current/.config, a file named kdeglobals showing up?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>
|
Working for me, Let's not hijack this thread and ping me in matrix. |
tox run -m lint
?tox run -e test-py310
? (supported versions:py39
,py310
,py311
,py312
)