Skip to content

feat(api)-Remove CLI functionality from SettingsService[#284]#291

Merged
youngtaekiim merged 1 commit intoeclipse-pullpiri:mainfrom
akshaylg0314:metric
Sep 24, 2025
Merged

feat(api)-Remove CLI functionality from SettingsService[#284]#291
youngtaekiim merged 1 commit intoeclipse-pullpiri:mainfrom
akshaylg0314:metric

Conversation

@akshaylg0314
Copy link
Contributor

📝 PR Description

->feat(api)-Remove CLI functionality from SettingsService[#284]
->Resolving Indentation error in MakeFile

🔗 Related Issue
closes->
#284

🧪 Test Method
CLI For setting service not avaliable and test settingservice,Nodeagent and monitoringserver.

✅ Checklist
[✅] Code conventions are followed
[✅] Tests are added/modified
[✅] Documentation is updated (if necessary)

Resolving Indentation error in MakeFile
@akshaylg0314
Copy link
Contributor Author

Hey @daeyoung-jeong-lge @youngtaekiim @leeeunkoo
please review the changes and merge this PR
Thanks!!

Copy link
Contributor

@daeyoung-jeong-lge daeyoung-jeong-lge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@daeyoung-jeong-lge
Copy link
Contributor

@leeeunkoo @youngtaekiim My review is done. Please merge this. Thanks.

@youngtaekiim youngtaekiim merged commit 20e42a6 into eclipse-pullpiri:main Sep 24, 2025
6 checks passed
yeohyeon-i added a commit to yeohyeon-i/pullpiri that referenced this pull request Sep 25, 2025
feat(api)-Remove CLI functionality from SettingsService[eclipse-pullpiri#284]
@leeeunkoo leeeunkoo linked an issue Oct 14, 2025 that may be closed by this pull request
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.

[TASK] Remove CLI functionality from SettingsService

3 participants