-
Notifications
You must be signed in to change notification settings - Fork 93
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
Configurable scope #670
Configurable scope #670
Conversation
c1465e5
to
8b3b51c
Compare
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
Still interested in merging this 😊. @gremo I can close it in case you believe is not necessary/useful. |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
Preventing auto-close |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
Can we get this reviewed by a maintainer? This is a very important feature that's missing now, and the PR has been open for months. (thanks @andreacioni for doing this!) |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
This PR was closed because it has been stalled for 10 days with no activity. |
Hello,
following discussion on issue #610 and #122 I tried to make a PR to address the need to have a configurable scope. I've also looked at #633 but I think the scope should be user defined and not changed to
TRANSIENT
by default.Please let me know if tests and/or sample are also needed.