Skip to content

Add a global variable 'ignored_message'#56

Closed
JustCodingYe wants to merge 1 commit intojaraco:mainfrom
JustCodingYe:main
Closed

Add a global variable 'ignored_message'#56
JustCodingYe wants to merge 1 commit intojaraco:mainfrom
JustCodingYe:main

Conversation

@JustCodingYe
Copy link
Copy Markdown

@JustCodingYe JustCodingYe commented Sep 28, 2024

Add a global variable 'ignored_message', which is used to expose the ignored message when a CSSStyleDeclaration: Unexpected token error occurs, allowing the user to retrieve that message for further processing, such as continuing to parse the ignored content.

…ignored message when a CSSStyleDeclaration: Unexpected token error occurs, allowing the user to retrieve that message.
@jaraco jaraco closed this Dec 23, 2024
@jaraco jaraco reopened this Dec 23, 2024
@jaraco jaraco changed the title Add a global variable 'ignored_message', which is used to expose the … Add a global variable 'ignored_message' Dec 23, 2024
@jaraco
Copy link
Copy Markdown
Owner

jaraco commented Dec 23, 2024

Thanks for the proposal. Unfortunately, I don't think this approach is sound. In general, global variables are dangerous. As a signal used as a public interface, they're even more dangerous.

What I recommend instead: file an issue describing your use-case and why it would be useful to know this value of logic. From there, we can assess the problem and explore different options for achieving your objective.

@jaraco jaraco closed this Dec 23, 2024
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