Skip to content

Conversation

@SamCartledge
Copy link
Collaborator

Sample

Adds git hooks to prevent committing API keys

Checklist

  • PR title follows convention - keyword: Short description of change
  • PR targets the correct branch
  • Self-review of changes
  • There are no warnings related to changes
  • A build was made and tested on all relevant platforms
  • No unrelated changes have been made to any other code or project files
  • No unnecessary includes or namespaces added
  • Code follows plugin coding style
  • New code and changed code has proper formatting
  • No unintentional formatting changes
  • Commits have descriptive titles

ArcGIS Maps SDK Version

@SamCartledge SamCartledge merged commit b21989a into main Nov 6, 2025
@SamCartledge SamCartledge deleted the sam/git-hooks branch November 6, 2025 00:07
mmgaw113 added a commit that referenced this pull request Nov 7, 2025
* Update readme.md

* Update readme.md

* Bug Fix: Add Script to enable camera & add to scenes (#191)

Add Script to enable camera & add to scenes

* Bring consistency to sample titles (#192)

* Bring consistency to sample titles

* more consistency

* Apply suggestions from code review

Co-authored-by: Rebekah Folsom <rfolsom@esri.com>

* Update sample_project/Assets/SampleViewer/Samples/MaterialByAttribute/readme.md

Co-authored-by: Rebekah Folsom <rfolsom@esri.com>

---------

Co-authored-by: Rebekah Folsom <rfolsom@esri.com>

* Update to 2022.3.62f2 (#194)

* Update to .62f2

* remove plugin path

* re-add HDRP, whoops

* Fix broken meta files, update HDRP

* Move ifdefs around

* fixes to URP, fix errors

* Update README.md

* internal: Add Git hooks to protect secrets (#193)

* Replace old hooks with internal hooks

* Fix line endings

* Create README.md

---------

Co-authored-by: and12279 <aren@esri.com>
Co-authored-by: Rebekah Folsom <rfolsom@esri.com>
Co-authored-by: Sam Cartledge <scartledge@esri.com>
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.

3 participants