Skip to content

chore(project_local): Allow to follow symlinks for projects configs#1891

Merged
olksdr merged 4 commits intomasterfrom
chore/follow-symlink-from-projects-dir
Mar 1, 2023
Merged

chore(project_local): Allow to follow symlinks for projects configs#1891
olksdr merged 4 commits intomasterfrom
chore/follow-symlink-from-projects-dir

Conversation

@olksdr
Copy link
Contributor

@olksdr olksdr commented Feb 28, 2023

fix: #1102

#skip-changelog

@olksdr olksdr requested a review from a team February 28, 2023 11:56
@olksdr olksdr self-assigned this Feb 28, 2023

if !entry.metadata().await?.is_file() {
let metadata = entry.metadata().await?;
if !(metadata.is_file() || metadata.is_symlink()) {
Copy link
Member

Choose a reason for hiding this comment

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

I don't want to be that guy, but: how would you feel about adding a test case?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done in fac37bd

Copy link
Member

Choose a reason for hiding this comment

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

Nice!

@olksdr olksdr merged commit dc47e21 into master Mar 1, 2023
@olksdr olksdr deleted the chore/follow-symlink-from-projects-dir branch March 1, 2023 12:50
jan-auer added a commit that referenced this pull request Mar 2, 2023
* master:
  doc(py): Add changelog entries (#1900)
  fix(build): Run check when PR is ready for review (#1899)
  chore(project_local): Allow to follow symlinks for projects configs (#1891)
  ref(project): Skip serializing default fields (#1887)
  chore(build): Run changelog check for draft PRs (#1897)
  chore(sentry): Add environment config option (#1890)
  feat(scrubbing): Scrub `span.data.http.query` with default scrubbers (#1889)
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.

Symbolic links are not loaded as projects

2 participants