Skip to content

Remove add_import script as support for package imports has been added upstream.#5056

Merged
Skarlso merged 8 commits intoeksctl-io:mainfrom
Skarlso:remove_import_script
Apr 6, 2022
Merged

Remove add_import script as support for package imports has been added upstream.#5056
Skarlso merged 8 commits intoeksctl-io:mainfrom
Skarlso:remove_import_script

Conversation

@Skarlso
Copy link
Copy Markdown
Contributor

@Skarlso Skarlso commented Apr 4, 2022

Description

Closes #4994

Support for adding a dot import has been added in ifacemaker. We pin to that specific version in the requirements.txt.

Checklist

  • Added tests that cover your change (if possible)
  • Added/modified documentation as required (such as the README.md, or the userdocs directory)
  • Manually tested
  • Made sure the title of the PR is a good description that can go into the release notes
  • (Core team) Added labels for change area (e.g. area/nodegroup) and kind (e.g. kind/improvement)

BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯

  • Backfilled missing tests for code in same general area 🎉
  • Refactored something and made the world a better place 🌟

@Skarlso Skarlso added the skip-release-notes Causes PR not to show in release notes label Apr 4, 2022
@Skarlso
Copy link
Copy Markdown
Contributor Author

Skarlso commented Apr 4, 2022

OH I have to update the build image eh?

@Skarlso
Copy link
Copy Markdown
Contributor Author

Skarlso commented Apr 4, 2022

Ok, pushed the images.

@Skarlso Skarlso enabled auto-merge (squash) April 6, 2022 08:18
@Skarlso Skarlso merged commit 23a23ac into eksctl-io:main Apr 6, 2022
SlevinWasAlreadyTaken pushed a commit to SlevinWasAlreadyTaken/eksctl that referenced this pull request Apr 11, 2022
…d upstream. (eksctl-io#5056)

* Remove add_import script as support for package imports has been added upstream.

* Update build image manifest, tag file and workflows

* Updated the build image manifest...

* Update build image manifest, tag file and workflows

* Updated mod file with proper version

* Update build image manifest, tag file and workflows
@hspencer77 hspencer77 mentioned this pull request Jul 8, 2022
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-release-notes Causes PR not to show in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Avoid maintaining a script for generating interfaces from structs

4 participants