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: Add getter methods for Scope class #3102

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

iKlotho
Copy link

@iKlotho iKlotho commented May 23, 2024

The getter methods would allow users to check if a specific tag/context/extra is already set or not. I have a use case like this and I don't want to access the protected members of a class. This PR adds getter methods for the following variables: _tags, _contexts, and _extras. If adding getter methods for _contexts and _extras doesn't make sense, I can remove them; however, I think they will be useful.

@iKlotho iKlotho changed the title Feat/add getter methods for Scope feat: Add getter methods for Scope class May 23, 2024
@antonpirker antonpirker added the Component: SDK Core Dealing with the core of the SDK label Jun 7, 2024
@antonpirker antonpirker added this to the SDK Core milestone Jun 7, 2024
@antonpirker antonpirker removed this from the SDK Core milestone Jun 20, 2024
@szokeasaurusrex
Copy link
Member

@iKlotho I would prefer to find an alternative solution to your problem, rather than add new public APIs to the Scope that we would need to maintain.

Can you please explain exactly why you need these getter methods?

@getsantry
Copy link

getsantry bot commented Aug 5, 2024

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot added the Stale label Aug 5, 2024
@getsantry
Copy link

getsantry bot commented Aug 27, 2024

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

3 similar comments
@getsantry
Copy link

getsantry bot commented Sep 17, 2024

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry
Copy link

getsantry bot commented Oct 9, 2024

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry
Copy link

getsantry bot commented Oct 31, 2024

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants