Skip to content

Conversation

@fuss86
Copy link
Contributor

@fuss86 fuss86 commented Aug 14, 2019

No description provided.

@fuss86 fuss86 force-pushed the SB-561 branch 2 times, most recently from 0c004c4 to 40b03d6 Compare August 18, 2019 10:14
@fuss86 fuss86 added ready ready to be merged and removed in progress labels Aug 18, 2019
@fuss86
Copy link
Contributor Author

fuss86 commented Aug 18, 2019

@carlspring please review :)

Every repository (with enabled indexing) has an index under the `strongbox-vault/storages/${storageId}/${repositoryId}/.index` directory
where the index is located.

* [Hosted](../knowledge-base/repositories.md#hosted) repositories have:
Copy link
Member

Choose a reason for hiding this comment

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

// same as above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@carlspring
Copy link
Member

@fuss86 : What is the status of this task? :)

@ghost
Copy link

ghost commented Sep 3, 2019

DeepCode Report (#b6b90a)

DeepCode analyzed this pull request.
There are no new issues.

@fuss86
Copy link
Contributor Author

fuss86 commented Sep 3, 2019

@steve-todorov could you re-review, please ?

@fuss86
Copy link
Contributor Author

fuss86 commented Sep 3, 2019

@fuss86 : What is the status of this task? :)

Close to be merged :) I fixed @steve-todorov comments

Copy link
Member

@steve-todorov steve-todorov left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks! :)

@steve-todorov steve-todorov merged commit e998a2d into master Sep 4, 2019
@carlspring carlspring deleted the SB-561 branch October 27, 2019 21:04
carlspring pushed a commit that referenced this pull request Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants