Skip to content

Normalize line endings for markdown files checked into git #6290

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

Merged
merged 1 commit into from
May 23, 2022

Conversation

dwyfrequency
Copy link
Contributor

Use https://git-scm.com/docs/gitattributes to normalize line endings for markdown files checked into git. We had issues in the past where files with different line endings would be checked in and the diff would show the entire file as changed. This will use the line endings specified for all markdown files checked in.

@changeset-bot
Copy link

changeset-bot bot commented May 23, 2022

⚠️ No Changeset found

Latest commit: 82f7e61

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • @firebase/analytics

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?18.9 kB? (?)
    esm5?23.5 kB? (?)
    main?24.6 kB? (?)
    module?18.9 kB? (?)
  • @firebase/analytics-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?2.57 kB? (?)
    esm5?2.80 kB? (?)
    main?3.24 kB? (?)
    module?2.57 kB? (?)
  • @firebase/api-documenter

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    main?3.95 kB? (?)
  • @firebase/app

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?13.8 kB? (?)
    esm5?18.0 kB? (?)
    main?19.0 kB? (?)
    module?13.8 kB? (?)
  • @firebase/app-check

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?25.2 kB? (?)
    esm5?29.8 kB? (?)
    main?31.0 kB? (?)
    module?25.2 kB? (?)
  • @firebase/app-check-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?2.27 kB? (?)
    esm5?2.48 kB? (?)
    main?2.94 kB? (?)
    module?2.27 kB? (?)
  • @firebase/app-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?5.28 kB? (?)
    esm5?6.13 kB? (?)
    lite?4.06 kB? (?)
    main?6.73 kB? (?)
    module?5.28 kB? (?)
  • @firebase/auth-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?20.2 kB? (?)
    esm5?27.0 kB? (?)
    main?29.6 kB? (?)
    module?20.2 kB? (?)
  • @firebase/auth/cordova

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?183 kB? (?)
    module?183 kB? (?)
  • @firebase/auth/internal

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?166 kB? (?)
    esm5?216 kB? (?)
    main?183 kB? (?)
    module?166 kB? (?)
  • @firebase/auth/react-native

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?168 kB? (?)
    module?168 kB? (?)
  • @firebase/component

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?6.54 kB? (?)
    esm5?8.61 kB? (?)
    main?8.99 kB? (?)
    module?6.54 kB? (?)
  • @firebase/database

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?248 kB? (?)
    esm5?276 kB? (?)
    main?281 kB? (?)
    module?248 kB? (?)
  • @firebase/database-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?18.0 kB? (?)
    esm5?21.1 kB? (?)
    main?21.8 kB? (?)
    module?18.0 kB? (?)
  • @firebase/database-compat/standalone

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    main?370 kB? (?)
  • @firebase/firestore

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?257 kB? (?)
    esm5?319 kB? (?)
    main?515 kB? (?)
    module?257 kB? (?)
    react-native?257 kB? (?)
  • @firebase/firestore-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?20.5 kB? (?)
    esm5?27.6 kB? (?)
    main?20.9 kB? (?)
    module?20.5 kB? (?)
    react-native?20.5 kB? (?)
  • @firebase/firestore-lite

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?80.5 kB? (?)
    esm5?96.2 kB? (?)
    main?135 kB? (?)
    module?80.5 kB? (?)
    react-native?80.7 kB? (?)
  • @firebase/functions

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?9.39 kB? (?)
    esm5?11.5 kB? (?)
    main?12.4 kB? (?)
    module?9.39 kB? (?)
  • @firebase/functions-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?1.79 kB? (?)
    esm5?1.98 kB? (?)
    main?2.34 kB? (?)
    module?1.79 kB? (?)
  • @firebase/installations

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?17.8 kB? (?)
    esm5?22.9 kB? (?)
    main?23.8 kB? (?)
    module?17.8 kB? (?)
  • @firebase/installations-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?944 B? (?)
    esm5?1.16 kB? (?)
    main?1.48 kB? (?)
    module?944 B? (?)
  • @firebase/logger

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    esm5?4.65 kB? (?)
    main?5.32 kB? (?)
    module?3.25 kB? (?)
  • @firebase/messaging

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?20.8 kB? (?)
    esm5?26.2 kB? (?)
    main?26.9 kB? (?)
    module?20.8 kB? (?)
  • @firebase/messaging-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?2.08 kB? (?)
    esm5?2.51 kB? (?)
    main?2.90 kB? (?)
    module?2.08 kB? (?)
  • @firebase/messaging-sw

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    main?29.6 kB? (?)
    module?22.9 kB? (?)
  • @firebase/performance

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?29.1 kB? (?)
    esm5?30.8 kB? (?)
    main?31.2 kB? (?)
    module?29.1 kB? (?)
  • @firebase/performance-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?1.10 kB? (?)
    esm5?1.37 kB? (?)
    main?1.69 kB? (?)
    module?1.10 kB? (?)
  • @firebase/remote-config

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?18.8 kB? (?)
    esm5?23.7 kB? (?)
    main?24.9 kB? (?)
    module?18.8 kB? (?)
  • @firebase/remote-config-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?1.85 kB? (?)
    esm5?2.76 kB? (?)
    main?3.09 kB? (?)
    module?1.85 kB? (?)
  • @firebase/rules-unit-testing

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    main?17.6 kB? (?)
  • @firebase/storage

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?54.5 kB? (?)
    esm5?60.5 kB? (?)
    main?57.3 kB? (?)
    module?54.5 kB? (?)
  • @firebase/storage-compat

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?5.67 kB? (?)
    esm5?8.41 kB? (?)
    main?8.82 kB? (?)
    module?5.67 kB? (?)
  • @firebase/util

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    browser?20.5 kB? (?)
    esm5?21.9 kB? (?)
    main?26.7 kB? (?)
    module?20.5 kB? (?)
  • @firebase/webchannel-wrapper

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    esm5?44.7 kB? (?)
    main?50.8 kB? (?)
    module?43.2 kB? (?)
  • bundle

    43 size changes

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    analytics (logEvent)?41.5 kB? (?)
    app-check (CustomProvider)?35.2 kB? (?)
    app-check (ReCaptchaEnterpriseProvider)?37.4 kB? (?)
    app-check (ReCaptchaV3Provider)?37.4 kB? (?)
    auth (Anonymous)?66.0 kB? (?)
    auth (EmailAndPassword)?70.1 kB? (?)
    auth (GoogleFBTwitterGitHubPopup)?89.9 kB? (?)
    auth (GooglePopup)?89.7 kB? (?)
    auth (GoogleRedirect)?89.9 kB? (?)
    auth (Phone)?76.1 kB? (?)
    database (Append to a list of data)?145 kB? (?)
    database (Filtering data)?144 kB? (?)
    database (Listen for child events)?160 kB? (?)
    database (Listen for value events + Detach listeners)?160 kB? (?)
    database (Listen for value events)?160 kB? (?)
    database (Read data once)?152 kB? (?)
    database (Save data as transactions)?162 kB? (?)
    database (Sort data)?146 kB? (?)
    database (Write data)?144 kB? (?)
    firestore (Persistence)?269 kB? (?)
    firestore (Query Cursors)?208 kB? (?)
    firestore (Query)?209 kB? (?)
    firestore (Read data once)?197 kB? (?)
    firestore (Realtime updates)?200 kB? (?)
    firestore (Transaction)?181 kB? (?)
    firestore (Write data)?181 kB? (?)
    firestore-lite (Query Cursors)?67.8 kB? (?)
    firestore-lite (Query)?71.0 kB? (?)
    firestore-lite (Read data once)?55.4 kB? (?)
    firestore-lite (Transaction)?80.0 kB? (?)
    firestore-lite (Write data)?65.2 kB? (?)
    functions (call)?29.1 kB? (?)
    messaging (send + receive)?44.7 kB? (?)
    performance (trace)?49.5 kB? (?)
    remote-config (getAndFetch)?43.9 kB? (?)
    storage (getBytes)?37.4 kB? (?)
    storage (getDownloadURL)?39.5 kB? (?)
    storage (getMetadata)?39.0 kB? (?)
    storage (list + listAll)?38.4 kB? (?)
    storage (updateMetadata)?39.2 kB? (?)
    storage (uploadBytes)?43.8 kB? (?)
    storage (uploadBytesResumable)?53.2 kB? (?)
    storage (uploadString)?44.0 kB? (?)

  • firebase

    29 size changes

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    firebase-analytics-compat.js?25.6 kB? (?)
    firebase-analytics.js?111 kB? (?)
    firebase-app-check-compat.js?22.7 kB? (?)
    firebase-app-check.js?90.0 kB? (?)
    firebase-app-compat.js?27.6 kB? (?)
    firebase-app.js?87.4 kB? (?)
    firebase-auth-compat.js?124 kB? (?)
    firebase-auth-cordova.js?472 kB? (?)
    firebase-auth-react-native.js?495 kB? (?)
    firebase-auth.js?417 kB? (?)
    firebase-compat.js?787 kB? (?)
    firebase-database-compat.js?166 kB? (?)
    firebase-database.js?604 kB? (?)
    firebase-firestore-compat.js?309 kB? (?)
    firebase-firestore-lite.js?267 kB? (?)
    firebase-firestore.js?837 kB? (?)
    firebase-functions-compat.js?8.12 kB? (?)
    firebase-functions.js?32.0 kB? (?)
    firebase-messaging-compat.js?37.5 kB? (?)
    firebase-messaging-sw.js?107 kB? (?)
    firebase-messaging.js?105 kB? (?)
    firebase-performance-compat.js?30.7 kB? (?)
    firebase-performance-standalone-compat.es2017.js?87.4 kB? (?)
    firebase-performance-standalone-compat.js?65.3 kB? (?)
    firebase-performance.js?123 kB? (?)
    firebase-remote-config-compat.js?27.1 kB? (?)
    firebase-remote-config.js?112 kB? (?)
    firebase-storage-compat.js?38.2 kB? (?)
    firebase-storage.js?146 kB? (?)

  • functions

    TypeBase (11d37ed)Merge (ffda3fd)Diff
    main?46 B? (?)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/9Tim6hwV5m.html

@google-oss-bot
Copy link
Contributor

@dwyfrequency dwyfrequency marked this pull request as ready for review May 23, 2022 20:43
@dwyfrequency dwyfrequency merged commit af7a182 into master May 23, 2022
@dwyfrequency dwyfrequency deleted the standardLineEndings branch May 23, 2022 22:06
@firebase firebase locked and limited conversation to collaborators Jun 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants