Skip to content
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

Rename factory package to preset #1334

Merged
merged 4 commits into from
Sep 11, 2023
Merged

Conversation

amirbenun
Copy link
Contributor

Summary of your changes

In this PR no logic has changed, only renaming of files, folders and functions

  1. Rename resources/fetching/factory package to preset.
  2. Rename functions New...Factory to New...Fetchers.
  3. FetchersMap and RegisteredFetcher moved from factory package to registry

Screenshot/Data

Related Issues

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary README/documentation (if appropriate)

@amirbenun amirbenun requested a review from a team as a code owner September 11, 2023 08:50
@mergify
Copy link

mergify bot commented Sep 11, 2023

This pull request does not have a backport label. Could you fix it @amirbenun? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v./d./d./d is the label to automatically backport to the 8./d branch. /d is the digit
    NOTE: backport-skip has been added to this pull request.

@github-actions
Copy link

github-actions bot commented Sep 11, 2023

📊 Allure Report - 💚 No failures were reported.

Result Count
🟥 Failed 0
🟩 Passed 39
⬜ Skipped 1

@@ -15,7 +15,7 @@
// specific language governing permissions and limitations
// under the License.

package factory
package preset
Copy link
Contributor

Choose a reason for hiding this comment

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

I have no idea for a better name here, maybe init? resources/fetching/init makes sense to me but I admit is not the most creative name.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

In go there's the init function so it might create a confusion.
I think preset describes well the idea of the package

@amirbenun amirbenun merged commit a552954 into elastic:main Sep 11, 2023
@amirbenun amirbenun deleted the rename-preset branch September 11, 2023 10:30
orestisfl pushed a commit to jeniawhite/cloudbeat that referenced this pull request Sep 18, 2023
jeniawhite added a commit that referenced this pull request Sep 22, 2023
* Implement Azure benchmark

* Fixing CR

* fixes

* [CloudFormation] Verify installation of cfn-signal (#1328)

* Bump magefile/mage-action from 2 to 3 in /.github/workflows (#1333)

Bumps [magefile/mage-action](https://github.com/magefile/mage-action) from 2 to 3.
- [Release notes](https://github.com/magefile/mage-action/releases)
- [Commits](magefile/mage-action@v2...v3)

---
updated-dependencies:
- dependency-name: magefile/mage-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* go.mod: Organize require blocks and update SDKs (#1330)

* Organize go.mod require blocks
* Update SDKs
* Remove go-errors/errors usage

* [CIS GCP] Add log bucket type to asset fetcher (#1327)

* add log bucket type

* bump cis policies version

* Rename factory package to preset (#1334)

* pre-commit: golangci-lint: Auto fix supported failures (#1336)

* Create a dedicated AWS organization preset (#1335)

* factory.NewCisAzureFactory

* Remove extra build step (#1337)

* AWS CSPM Resources add ECS data (#1312)

* Populate vulnerability.reference with a link to NVD (#1303)

* opulate vulnerability.reference with a link to NVD

* revert the if statemenet

* [Cloud Security] [Telemetry] fix package policy vars to use posture field (#1325)

fix package policy vars to use posture field

* Use branches to get bundles (#1332)

* Refactor to be compatible with main and fix compilation

* Fixing linter

* Adding unit tests

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Orestis Floros <[email protected]>
Co-authored-by: Amir Ben Nun <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Uri Weisman <[email protected]>
Co-authored-by: Or Ouziel <[email protected]>
Co-authored-by: Maxim Kholod <[email protected]>
Co-authored-by: Lola <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants