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

Add recipe for processing-3-mode #7323

Closed
wants to merge 3 commits into from
Closed

Add recipe for processing-3-mode #7323

wants to merge 3 commits into from

Conversation

motform
Copy link
Contributor

@motform motform commented Dec 31, 2020

Brief summary of what the package does

processing-3-mode is a minimalist mode for the third version of the Processing programming language. The processing-mode packade is more sophisticated/opinionated, but not updated for p3. This package has simpler ambitions, which it fulfils by leveraging the Processing command line tools.

Direct link to the package repository

https://github.com/motform/processing-3-mode

Your association with the package

I am the maintainer and creator.

Relevant communications with the upstream package maintainer

None needed.

Checklist

Please confirm with x:

  • The package is released under a GPL-Compatible Free Software License.
  • I've read CONTRIBUTING.org
  • I've used the latest version of package-lint to check for packaging issues, and addressed its feedback
  • My elisp byte-compiles cleanly
  • M-x checkdoc is happy with my docstrings
  • I've built and installed the package using the instructions in CONTRIBUTING.org
  • I have confirmed some of these without doing them

@tarsius
Copy link
Member

tarsius commented Dec 31, 2020

Please fix this issue:

Recipe ’processing-3-mode’ contains mismatched package name ’arduino-cli-mode’

@motform
Copy link
Contributor Author

motform commented Dec 31, 2020

Please fix this issue:

Recipe ’processing-3-mode’ contains mismatched package name ’arduino-cli-mode’

That's what you get for copy-pasting while making New Years dinner. 😅 It's fixed now!

@riscy
Copy link
Member

riscy commented Jan 10, 2021

Re: the existing process-mode --

is more sophisticated/opinionated, but not updated for p3

is this update something that you see can be fixed? If it's feasible, that would be far better for users. Maybe the ways in which it is opinionated can be turned into user options as well. I don't see a lot of recent GitHub activity from @ptrv but I will leave this mention.

In the meantime, processing-3-mode.el#L170: Terminate auto-mode-alist entries with \\'

@riscy
Copy link
Member

riscy commented Feb 20, 2021

Hello, I just thought I'd ping and check if there's a followup. Maybe my suggestion isn't possible, and we also haven't heard back from @ptrv so this is probably fine. It looks like the auto-mode-alist entry still needs to be fixed.

@riscy riscy added the awaiting-upstream Awaiting action from an upstream maintainer label Feb 20, 2021
@motform
Copy link
Contributor Author

motform commented Feb 22, 2021

Hello, I just thought I'd ping and check if there's a followup. Maybe my suggestion isn't possible, and we also haven't heard back from @ptrv so this is probably fine. It looks like the auto-mode-alist entry still needs to be fixed.

Hi, I have been busy with other things, but yes, in essence I agree with the your previous post that it would be much nicer if there was one Processing mode. Having version-defined modes is not very useful, esp. considering the fact that they are about to release P4, whose breaking changes are completely ignored by any type of editor mode.

I looked over @ptrv's mode and we more or less do the same things, sans how we handle the calls to the compiler (CLI front end vs path to the compiler). I think it would be optimal to patch/fork his mode with support for the latter method and maybe call it "Processing mode". I have sent him an email, will keep this thread updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-upstream Awaiting action from an upstream maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants