react-native-windows icon indicating copy to clipboard operation
react-native-windows copied to clipboard

Standardize hermes packaging

Open mganandraj opened this issue 4 years ago • 4 comments
trafficstars

Problem Description

Package is unsigned, not named based on convention and uploaded from private credentials.

Steps To Reproduce

Notice the packages.config

Expected Results

Hermes package is

  1. Named as Microsoft.*
  2. Signed
  3. Published by proper user agents

CLI version

npx react-native --version

Environment

npx react-native info

Target Platform Version

No response

Target Device(s)

No response

Visual Studio Version

No response

Build Configuration

No response

Snack, code example, screenshot, or link to a repository

No response

mganandraj avatar Nov 09 '21 19:11 mganandraj

@mganandraj FYI in the future you can use either the feature request template or the empty issue template for when the bug template is a bad fit.

chrisglein avatar Nov 11 '21 17:11 chrisglein

Multiple issues tracked here. This one would be a change to the template that we would want to frontload:

not named based on convention

There's some setup in this repo for signing that may not be available in Hermes repo (https://github.com/microsoft/hermes-windows). So where this work happens will affect how straightforward this is.

chrisglein avatar Nov 15 '21 19:11 chrisglein

@dedavis6797 Is this on your radar for the Hermes work? With RN 0.69 including Hermes, is this the right time to get our version of Hermes synced nicely with RNW 0.69?

Can you assign this to the right person?

jonthysell avatar May 06 '22 23:05 jonthysell

@dedavis6797 Does this need to be addressed for the Hermes rollout in 0.70?

chrisglein avatar Aug 30 '22 17:08 chrisglein

Flagging to discuss if issue should remain prioritized / what are next steps as issue has been punted 2 releases.

chiaramooney avatar Jan 09 '23 19:01 chiaramooney

Is now being signed properly. Name change is the only remaining part of this. (will also involve making sure the account it's published under can publish that) People who probably best know how to do this: @dannyvv and @jonthysell

Name: Microsoft.Hermes Question: Do we need separate for Win32 vs. UWP? Today we only release for UWP. May change in the future.

@vmoroz is going to take a look to get the first steps started

chrisglein avatar Jan 12 '23 19:01 chrisglein