Skip to content
This repository has been archived by the owner on Jul 12, 2024. It is now read-only.

character warning limit (ready for testing) #215

Closed
wants to merge 15 commits into from

Conversation

SCBionicle
Copy link
Contributor

@SCBionicle SCBionicle commented Nov 16, 2020

Fixes #101

An implementation of the character limit warning. I'm unfamiliar with visual studio extension API, especially when it comes to the error list window. I got a borked installation which is preventing me from compiling, if someone else could if this works, not asking for debugging, though, if you don't want to.

@SCBionicle SCBionicle marked this pull request as draft November 16, 2020 22:20
@SCBionicle
Copy link
Contributor Author

If anyone's wondering, I was was using this as reference, but it provides an incomplete picture.

https://gist.github.com/DinisCruz/3185313

@SCBionicle SCBionicle marked this pull request as ready for review January 24, 2021 19:19
@SCBionicle SCBionicle changed the title character warning limit (wip) character warning limit (ready for testing) Feb 21, 2021
@SCBionicle
Copy link
Contributor Author

If I may ask, why was this closed?

@malware-dev
Copy link
Owner

malware-dev commented Mar 31, 2024

Well, ignoring the fact that it was a completely untested thing by your own admission... the more important thing is #308.

MDK1 is for all intents and purposes dead. MDK2 is underway.

@SCBionicle
Copy link
Contributor Author

I understand. Sucks that Microsoft broke your functionality. I might have a few private projects I could share with you to help develop the minifier.

P.S. It's been a while since I work on this, but I could have sworn I tested it and was waiting for approval. Oh well.

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

Successfully merging this pull request may close these issues.

[Suggestion] Warning when deploying code that goes over SE's character limit
2 participants