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

[Feature] Profile #5

Closed
kxxt opened this issue May 8, 2024 · 5 comments · Fixed by #29
Closed

[Feature] Profile #5

kxxt opened this issue May 8, 2024 · 5 comments · Fixed by #29
Milestone

Comments

@kxxt
Copy link
Owner

kxxt commented May 8, 2024

Allow users to load settings from ~/.config/tracexec/config.toml.

@kxxt
Copy link
Owner Author

kxxt commented May 14, 2024

It need to be documented that the config format might be changed in an incompatible way between minor versions since we haven't reached 1.0 yet.

@kxxt kxxt added this to the 0.5 milestone May 26, 2024
@kxxt kxxt self-assigned this May 26, 2024
@GeorgeMuscat
Copy link

Hey, I was looking at possibly having a go at implementing this. Is this project open to PRs?

@kxxt
Copy link
Owner Author

kxxt commented Jun 9, 2024

Hey, I was looking at possibly having a go at implementing this. Is this project open to PRs?

Yes. PRs are welcome. ☺️

@kxxt kxxt removed their assignment Jun 9, 2024
@kxxt
Copy link
Owner Author

kxxt commented Jul 14, 2024

Hi @GeorgeMuscat, are you still interested in implementing this feature 🤗? If not, I might do it in the next few days.

@GeorgeMuscat
Copy link

I had a weak attempt but realised I wasn't familiar enough with the project and the design decisions to do it properly.

Might be best if you implement it. I will watch this issue and will have a look at your PR when you're done so I can get an idea of how you would want this solved. Thanks!

@kxxt kxxt mentioned this issue Jul 15, 2024
7 tasks
@kxxt kxxt closed this as completed in #29 Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants