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

elisp: Ignore more build tools #4263

Merged
merged 1 commit into from
Feb 19, 2025
Merged

elisp: Ignore more build tools #4263

merged 1 commit into from
Feb 19, 2025

Conversation

jcs090218
Copy link
Contributor

.eask, .eldev, .keg are other build tools like .cask. dist is the default folder to build package distribution files.

Reasons for making this change:

The reason is pretty much the same as #1096.

Links to documentation supporting these rule changes:

.eask, .eldev, .keg are other build tools like .cask.
dist is the default folder to build package distribution files.
@jcs090218
Copy link
Contributor Author

From #3567 (comment).

@martinwoodward Can you help review this PR? Thanks!

@jcs090218
Copy link
Contributor Author

Close this due to no response.

@jcs090218 jcs090218 closed this Jun 14, 2024
@jcs090218 jcs090218 reopened this Feb 19, 2025
@jcs090218
Copy link
Contributor Author

jcs090218 commented Feb 19, 2025

@thorrsson Can you review this? Thanks!

Copy link
Collaborator

@thorrsson thorrsson left a comment

Choose a reason for hiding this comment

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

I'm not super familiar with Cask and it's workings but from the docs links this looks reasonable to me.

@thorrsson thorrsson merged commit 471141b into github:main Feb 19, 2025
@lukasito69
Copy link

lukasito69 commented Feb 19, 2025 via email

@jcs090218 jcs090218 deleted the elisp branch February 19, 2025 17:19
@Dayat03
Copy link

Dayat03 commented Feb 19, 2025 via email

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

Successfully merging this pull request may close these issues.

4 participants