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

feat(daemon): Rename Locator to Config, improve type exports #2212

Merged
merged 3 commits into from
Apr 15, 2024

Conversation

rekmarks
Copy link
Contributor

Renames the daemon's Locator, used to specify its file and storage paths, to Config to get rid of the name collision with endo locator strings. Also adds return values to functions in exported types, and removes redundant definition of Config type.

@rekmarks rekmarks requested a review from kriskowal April 15, 2024 21:39
@rekmarks rekmarks enabled auto-merge April 15, 2024 21:39
@kriskowal
Copy link
Member

There is also a touch-point in cli: the file named locator.js exports effectively the config.

@rekmarks rekmarks force-pushed the rekmarks-old-locator-is-now-config branch from fd08ae4 to d5d9bc6 Compare April 15, 2024 22:09
Renames the daemon's `Locator`, used to specify its file and storage
paths, to `Config` to get rid of the name collision with endo locator
strings.
Adds return values for exported functions in `./types.d.ts`, and removes
redundant definition of `Config` type.
@rekmarks rekmarks force-pushed the rekmarks-old-locator-is-now-config branch from d5d9bc6 to 89d4a2c Compare April 15, 2024 22:26
@rekmarks rekmarks merged commit b71d7d1 into master Apr 15, 2024
18 checks passed
@rekmarks rekmarks deleted the rekmarks-old-locator-is-now-config branch April 15, 2024 22:33
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.

2 participants