Skip to content

Conversation

mimartin12
Copy link

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Patches the Sed command to handle the new Android manifest format so that we can release.

Code changes

  • .github/workflows/release.yml: Updated sed command.

Before you submit

  • I have added unit tests where it makes sense to do so (encouraged but not required)
  • This change requires a documentation update (notify the documentation team)
  • This change has particular deployment requirements (notify the DevOps team)

@mimartin12 mimartin12 self-assigned this Feb 10, 2022
@mimartin12 mimartin12 merged commit 9eed421 into master Feb 10, 2022
@mimartin12 mimartin12 deleted the patch-release-sed branch February 10, 2022 16:42
joseph-flinn pushed a commit that referenced this pull request Feb 10, 2022
@mpbw2
Copy link
Contributor

mpbw2 commented Feb 10, 2022

Any chance this can be changed to use an XML parser instead of sed? Asking because I'm not sure how/when some of the recent formatting changes occurred, and they'll probably be undone/fixed which will break this change. Tag parsing would help bullet proof against formatting shenanigans going forward.

@mimartin12
Copy link
Author

I would love to move towards an XML parser. I have added it to our list for the retro.

fedemkr added a commit that referenced this pull request Feb 10, 2022
* master:
  Updated Sed expression for Android manifests (#1763)
  Bumped version to 2.16.1 (#1762)
  Add iOS Share Extension to our version bump automation (#1761)
@vvolkgang
Copy link
Member

@mimartin12 we previously discussed about CAKE, this would be one of the usecases for it. This PR would be another good one.

fedemkr added a commit that referenced this pull request Feb 11, 2022
* master: (34 commits)
  check email for null before authenticating (#1769)
  remove datepicker style workaround (#1768)
  Bump version to 2.16.2 (#1765)
  Improved code for periodic Autofocus on scan for better cancellation and task handlilng (#1764)
  Updated Sed expression for Android manifests (#1763)
  Bumped version to 2.16.1 (#1762)
  Add iOS Share Extension to our version bump automation (#1761)
  Bumped version to 2.16.0 (#1760)
  Improved Autofocus code on ScanPage for better cancellation and exception handling #1228 (#1759)
  [Help] Update links to new pattern (#1758)
  Client & Version headers (#1757)
  Autosync the updated translations (#1752)
  Fix delete account SSO with CME that the OTP parameter was being sent incorrectly to the server (#1751)
  [Icons] Fast follower changes (#1750)
  Removed punctuation on some string resources regarding send (#1747)
  Set background of biometricCell from theme (#1744)
  Cleared About information about push registration, just left it in DEBUG mode (#1742)
  Cleared Console.WriteLine for iOS push notifications issue (#1739)
  Autosync the updated translations (#1734)
  Update FA icons on LockPasswordViewController (#1729)
  ...
fedemkr added a commit that referenced this pull request Feb 14, 2022
* master:
  [Icons] - BUG - Update groupings icon for collections (#1773)
  Autosync the updated translations (#1766)
  check email for null before authenticating (#1769)
  remove datepicker style workaround (#1768)
  Bump version to 2.16.2 (#1765)
  Improved code for periodic Autofocus on scan for better cancellation and task handlilng (#1764)
  Updated Sed expression for Android manifests (#1763)
  Bumped version to 2.16.1 (#1762)
  Add iOS Share Extension to our version bump automation (#1761)
  Bumped version to 2.16.0 (#1760)
  Improved Autofocus code on ScanPage for better cancellation and exception handling #1228 (#1759)
  [Help] Update links to new pattern (#1758)
  Client & Version headers (#1757)
  Autosync the updated translations (#1752)
  Fix delete account SSO with CME that the OTP parameter was being sent incorrectly to the server (#1751)
  [Icons] Fast follower changes (#1750)
  Removed punctuation on some string resources regarding send (#1747)

# Conflicts:
#	src/Android/Properties/AndroidManifest.xml
#	src/iOS.Autofill/Info.plist
#	src/iOS.Extension/Info.plist
#	src/iOS.ShareExtension/Info.plist
#	src/iOS/Info.plist
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.

4 participants