Skip to content

Remove atomicwrites dependency#332

Merged
ofek merged 3 commits into
masterfrom
a
Jul 9, 2022
Merged

Remove atomicwrites dependency#332
ofek merged 3 commits into
masterfrom
a

Conversation

@ofek
Copy link
Copy Markdown
Contributor

@ofek ofek commented Jul 9, 2022

@ofek
Copy link
Copy Markdown
Contributor Author

ofek commented Jul 9, 2022

differences:

  • no directory fsync after temp file fsync b/c I'm not sure if that's necessary
  • no special macOS logic

Comment thread src/hatch/config/user.py Outdated
Comment thread src/hatch/config/utils.py Outdated
ofek and others added 2 commits July 9, 2022 08:39
Co-authored-by: Thomas Grainger <tagrain@gmail.com>
@ofek ofek merged commit 53598d3 into master Jul 9, 2022
@ofek ofek deleted the a branch July 9, 2022 14:50
github-actions Bot pushed a commit that referenced this pull request Jul 9, 2022
* Remove `atomicwrites` dependency

* Apply suggestions from code review

Co-authored-by: Thomas Grainger <tagrain@gmail.com>

* apple...

Co-authored-by: Thomas Grainger <tagrain@gmail.com> 53598d3
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.

2 participants