Jason Gallavin

Results 48 comments of Jason Gallavin

@seawerst The part that I'm thinking about is how many entries for a property should be inlined? CSS properties can be overridden many times. Including them all could cause the...

I am definitely interested. This library doesn't seem to be a huge time sink so I can definitely help keep the library updated. When you say you recall working on...

I'll start familiarizing myself with the codebase and the unfinished work over the week end. I want to double check everything before making a release and potentially messing something up...

My apologies, that was it. The repository appeared to show more access but I was mistaken since I hadn't accepted the invite yet. What is your opinion of me doing...

I noticed that all of the classes are marked as public. If someone uses the library in a way the readme doesn't state I could accidentally introduce a breaking change...

I can't merge PR's created by me since it requires one approved review but I can't review my own PR. I was able to merge some dependabot PR's however.

Thanks for making that adjustment! I have been going through the issues and closed ones I confirmed are already completed. I did notice [this issue](https://github.com/milkshakesoftware/PreMailer.Net/issues/157) that was not marked as...

Oh wow, I usually catch large diffs like that. I must have had ignore whitespace checked on my diff. It's probably a line ending issue since it appears on lines...

@martinnormark I think I found the root of the issue. It seems that the line endings of the files in the git index are inconsistent. My git configuration is set...

> @jasekiw I was excited when I saw this, I was hoping for an update soon. This is a great project. Hi @TopCoder02 I should have some time this week...