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

chore(repo): add git attributes configuration #29584

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

rwaskiewicz
Copy link
Contributor

@rwaskiewicz rwaskiewicz commented Jun 4, 2024

Issue number: N/A


What is the current behavior?

What is the new behavior?

add a .gitattributes configuration file to the repo. this will allow users of unix and windows machines to be able to commit to the repo without worrying about line endings.

previously, running the lint npm task in the core directory would cause the line endings to change locally. with this change, line endings should not change

Does this introduce a breaking change?

  • Yes
  • No

Other information

add a .gitattributes configuration file to the repo. this will allow
users of unix and windows machines to be able to commit to the repo
without worrying about line endings.

previously, running the `lint` npm task in the core directory would
cause the line endings to change locally. with this change, line endings
should not change
Copy link

vercel bot commented Jun 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ionic-framework ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 4, 2024 8:01pm

@github-actions github-actions bot added the package: core @ionic/core package label Jun 4, 2024
@rwaskiewicz rwaskiewicz marked this pull request as ready for review June 4, 2024 19:53
@rwaskiewicz rwaskiewicz requested a review from a team as a code owner June 4, 2024 19:53
@rwaskiewicz rwaskiewicz requested a review from thetaPC June 4, 2024 19:53
@rwaskiewicz rwaskiewicz enabled auto-merge June 4, 2024 19:57
@rwaskiewicz rwaskiewicz added this pull request to the merge queue Jun 4, 2024
Merged via the queue into main with commit 34d4055 Jun 4, 2024
49 checks passed
@rwaskiewicz rwaskiewicz deleted the rw/gitattributes branch June 4, 2024 20:12
rwaskiewicz added a commit that referenced this pull request Jun 5, 2024
this was mistakenly put in the core directory in #29584
github-merge-queue bot pushed a commit that referenced this pull request Jun 5, 2024
Issue number: resolves N/A

---------

<!-- Please do not submit updates to dependencies unless it fixes an
issue. -->

<!-- Please try to limit your pull request to one type (bugfix, feature,
etc). Submit multiple pull requests if needed. -->

## What is the current behavior?
<!-- Please describe the current behavior that you are modifying. -->

## What is the new behavior?
<!-- Please describe the behavior or changes that are being added by
this PR. -->

this was mistakenly put in the core directory in
#29584

## Does this introduce a breaking change?

- [ ] Yes
- [X] No

<!--
  If this introduces a breaking change:
1. Describe the impact and migration path for existing applications
below.
  2. Update the BREAKING.md file with the breaking change.
3. Add "BREAKING CHANGE: [...]" to the commit description when merging.
See
https://github.com/ionic-team/ionic-framework/blob/main/docs/CONTRIBUTING.md#footer
for more information.
-->


## Other information

<!-- Any other information that is important to this PR such as
screenshots of how the component looks before and after the change. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: core @ionic/core package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants