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

Open to a formatting fixup PR? #120

Open
TRSx80 opened this issue Jan 23, 2023 · 1 comment
Open

Open to a formatting fixup PR? #120

TRSx80 opened this issue Jan 23, 2023 · 1 comment

Comments

@TRSx80
Copy link

TRSx80 commented Jan 23, 2023

As I was working on #119, I noticed that there is a bit of weird indentation here and there, some extra whitespace, etc.

Now that I think I got the hang of this, there are a few more packages I will probably eventually submit PRs to support.

But that would be easier if I could use Lispy freely as I normally do when editing ELisp (which I do quite a lot of). The problem is that Lispy will automatically reformat things to proper ELisp conventions as you go along, for example fixing indentation when you hit RET at end of line, etc. Once I realized what had happened, I had to turn it off and undo lest I wreck the current (incorrect) formatting of some (large?) part of the file nord-theme.el.

So I thought maybe it would be best to submit 1 PR first cleaning up the formatting, before proceeding. I mean, if that sounds like a good idea to you?

What do you think?

@TRSx80
Copy link
Author

TRSx80 commented Jan 26, 2023

After opening this issue, I was looking around a bit more and I started to realize that you seem to be a bit overwhelmed with the volume of issues and PRs, etc.

I am not bringing this up to harangue you about that, quite the contrary. And congratulations on the success of the project. 😉

However I think I shall proceed by simply sending separate PRs for whatever packages I have in mind (so far Gnus and kpm-list, maybe others later). That way people can just apply those patches (independently if they wish) until you can get caught up or get some help or whatever.

I guess I leave it up to you if you want to keep this issue open I am happy to work in it later whenever you are ready. Or close it for the time being.

Cheers!

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

No branches or pull requests

1 participant