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

Don't require Lamdera during project setup #791

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JEphron
Copy link

@JEphron JEphron commented Nov 18, 2022

While it's nice to make accommodations for users who use the Lamdera platform, the primary purpose of this plugin is support for the Elm language. Most likely the majority of users have never even heard of Lamdera, and shouldn't be required to learn about it or install it to have support for Elm in their editor. :)

This PR also bumps the max Intellij version so that we can use this plugin in Intellij 2022.

@Aarkon
Copy link

Aarkon commented Nov 24, 2022

Thank you for the work, I can confirm that it works (checked with IntelliJ IDEA 2022.2.4 (Community Edition))!

Copy link

@hewison-chris hewison-chris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@paulwwisl
Copy link

paulwwisl commented May 4, 2023

Already not good enough for IDEA 2022.3.3

image

update pluginUntilBuild property in gradle.properties.
pluginUntilBuild = 999.*

@cies
Copy link
Contributor

cies commented Apr 3, 2024

I'm going to merge this. LGTM.

@cies cies added the feature label Apr 3, 2024
@cies cies added this to the 5.0.1 milestone Apr 4, 2024
@cies cies self-assigned this Apr 4, 2024
@cies
Copy link
Contributor

cies commented Apr 10, 2024

@JEphron Could you merge master into this branch? I've fixed CI on master. Thanks.

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

Successfully merging this pull request may close these issues.

None yet

5 participants