-
Notifications
You must be signed in to change notification settings - Fork 2k
fix(utils): save config #578
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
Conversation
All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
recheck |
a83d9d7
to
c6cd76c
Compare
89a5260
to
335f3f5
Compare
335f3f5
to
62352a7
Compare
62352a7
to
3f71a1d
Compare
Nice one, thank you for contributing this fix. It looks like there are CI failures for formatting
could you take a look and once resolved then I will merge. |
3f71a1d
to
076b5a5
Compare
Hi @tibo-openai 👋 Thank you for the feedback! I’ve resolved the Prettier formatting issues. However, I noticed that the current |
Merged #617, thank you! |
Simplified the code a bit. Thank you for the contribution! |
Description
When
saveConfig
is called, the project doc is incorrectly saved into user instructions. This change ensures that only user instructions are saved toinstructions.md
during saveConfig, preventing data corruption.close: #576