Browse Source

style(js): Always ` ` whitespace after comments (#26732)

Evan Purkhiser 3 years ago
parent
commit
bbae4b6354

+ 1 - 1
static/app/actionCreators/modal.tsx

@@ -191,7 +191,7 @@ export type SentryAppDetailsModalOptions = {
   isInstalled: boolean;
   onInstall: () => Promise<void>;
   organization: Organization;
-  onCloseModal?: () => void; //used for analytics
+  onCloseModal?: () => void; // used for analytics
 };
 
 type DebugFileSourceModalOptions = {

+ 1 - 1
static/app/components/assistant/getGuidesContent.tsx

@@ -316,7 +316,7 @@ function getDemoModeGuides(): GuidesContent {
     {
       guide: 'sidebar',
       requiredTargets: ['projects', 'issues'],
-      priority: 1, //lower number means higher priority
+      priority: 1, // lower number means higher priority
       markOthersAsSeen: true,
       steps: [
         {

+ 1 - 1
static/app/components/assistant/guideAnchor.tsx

@@ -20,7 +20,7 @@ import space from 'app/styles/space';
 import theme from 'app/utils/theme';
 
 type Props = {
-  target?: string; //Shouldn't target be mandatory?
+  target?: string; // Shouldn't target be mandatory?
   position?: React.ComponentProps<typeof Hovercard>['position'];
   offset?: string;
   to?: {

+ 1 - 1
static/app/components/asyncComponent.tsx

@@ -301,7 +301,7 @@ export default class AsyncComponent<
         return state;
       },
       () => {
-        //if everything is loaded and we don't have an error, call the callback
+        // if everything is loaded and we don't have an error, call the callback
         if (this.state.remainingRequests === 0 && !this.state.error) {
           this.onLoadAllEndpointsSuccess();
         }

+ 2 - 2
static/app/components/contextPickerModal.tsx

@@ -246,7 +246,7 @@ class ContextPickerModal extends Component<Props> {
     });
   };
 
-  //TODO(TS): Fix typings
+  // TODO(TS): Fix typings
   customOptionProject = ({label, ...props}: any) => {
     const project = this.props.projects.find(({slug}) => props.value === slug);
     if (!project) {
@@ -278,7 +278,7 @@ class ContextPickerModal extends Component<Props> {
     if (integrationConfigs.length) {
       return t('Select a configuration to continue');
     }
-    //if neither project nor org needs to be selected, nothing will render anyways
+    // if neither project nor org needs to be selected, nothing will render anyways
     return '';
   }
 

+ 2 - 2
static/app/components/demo/demoHeader.tsx

@@ -79,7 +79,7 @@ export default function DemoHeader() {
   );
 }
 
-//Note many of the colors don't come from the theme as they come from the marketing site
+// Note many of the colors don't come from the theme as they come from the marketing site
 const Wrapper = styled('div')<{collapsed: boolean}>`
   padding-right: ${space(3)};
   background-color: ${p => p.theme.white};
@@ -117,7 +117,7 @@ const BaseButton = styled(Button)`
   text-transform: uppercase;
 `;
 
-//Note many of the colors don't come from the theme as they come from the marketing site
+// Note many of the colors don't come from the theme as they come from the marketing site
 const GetStarted = styled(BaseButton)`
   border-color: transparent;
   box-shadow: 0 2px 0 rgb(54 45 89 / 10%);

+ 1 - 1
static/app/components/dropdownAutoComplete/autoCompleteFilter.tsx

@@ -46,7 +46,7 @@ function autoCompleteFilter(
   }
 
   if (hasRootGroup(items)) {
-    //if the first item has children, we assume it is a group
+    // if the first item has children, we assume it is a group
     return flatMap(filterGroupedItems(items, inputValue), item => {
       const groupItems = item.items.map(groupedItem => ({
         ...groupedItem,

+ 1 - 1
static/app/components/eventOrGroupHeader.tsx

@@ -101,7 +101,7 @@ class EventOrGroupHeader extends Component<Props> {
             query: {
               query: this.props.query,
               ...(location.query.sort !== undefined ? {sort: location.query.sort} : {}), // This adds sort to the query if one was selected from the issues list page
-              ...(location.query.project !== undefined ? {} : {_allp: 1}), //This appends _allp to the URL parameters if they have no project selected ("all" projects included in results). This is so that when we enter the issue details page and lock them to a project, we can properly take them back to the issue list page with no project selected (and not the locked project selected)
+              ...(location.query.project !== undefined ? {} : {_allp: 1}), // This appends _allp to the URL parameters if they have no project selected ("all" projects included in results). This is so that when we enter the issue details page and lock them to a project, we can properly take them back to the issue list page with no project selected (and not the locked project selected)
             },
           }}
           onClick={onClick}

+ 1 - 1
static/app/components/events/interfaces/debugMeta-v2/debugImageDetails/candidate/information/processingIcon.tsx

@@ -35,7 +35,7 @@ function ProcessingIcon({processingInfo}: Props) {
           new Error('Unknown image candidate ProcessingIcon status')
         );
       });
-      return null; //this shall never happen
+      return null; // this shall never happen
     }
   }
 }

+ 1 - 1
static/app/components/events/interfaces/stacktraceContent.tsx

@@ -222,7 +222,7 @@ class StacktraceContent extends React.Component<Props, State> {
             onAddressToggle={this.handleToggleAddresses}
             image={image}
             maxLengthOfRelativeAddress={maxLengthOfAllRelativeAddresses}
-            registers={{}} //TODO: Fix registers
+            registers={{}} // TODO: Fix registers
             isFrameAfterLastNonApp={isFrameAfterLastNonApp}
             includeSystemFrames={includeSystemFrames}
             onFunctionNameToggle={this.handleToggleFunctionName}

Some files were not shown because too many files changed in this diff